@jsenv/navi 0.29.504 → 0.29.506

package/dist/jsenv_navi.js

@@ -36601,18 +36601,18 @@ const useContentKeepingLeavingPages = (branch, readShownBranch, parentPage) => {
       }
       entry.hidden = true;
       entry.page.leavingSignal.value = true;
-      hideNodesBetween(start, end);
       // The page renders on its own while kept (a component reading a signal
       // the address moved), never through this container: a top node it
-      // replaces lands between the markers unmarked. Its mutation is heard
-      // before the next frame.
+      // replaces lands between the markers unmarked, a text it writes shows
+      // again. Both are heard before the next frame.
       const observer = new MutationObserver(() => {
         if (entry.hold.released) {
           observer.disconnect();
           return;
         }
-        hideNodesBetween(start, end);
+        hideNodesBetween(start, end, observer);
       });
+      hideNodesBetween(start, end, observer);
       observer.observe(start.parentNode, {
         childList: true
       });
@@ -36656,18 +36656,28 @@ const useContentKeepingLeavingPages = (branch, readShownBranch, parentPage) => {
   };
 };
 const countRenders = count => count + 1;
-const hideNodesBetween = (start, end) => {
+const hideNodesBetween = (start, end, observer) => {
   const focusedElement = document.activeElement;
   let node = start.nextSibling;
   while (node && node !== end) {
-    if (node.nodeType === Node.ELEMENT_NODE && !node.hasAttribute(LEAVING_ATTRIBUTE)) {
-      node.setAttribute(LEAVING_ATTRIBUTE, "");
-      // What the keyboard held in it goes where the page's removal would
-      // have sent it: a hidden element keeps the focus, and would go on
-      // receiving the keys pressed during the movement.
-      if (node.contains(focusedElement)) {
-        focusedElement.blur();
-      }
+    if (node.nodeType === Node.ELEMENT_NODE) {
+      if (!node.hasAttribute(LEAVING_ATTRIBUTE)) {
+        node.setAttribute(LEAVING_ATTRIBUTE, "");
+        // What the keyboard held in it goes where the page's removal would
+        // have sent it: a hidden element keeps the focus, and would go on
+        // receiving the keys pressed during the movement.
+        if (node.contains(focusedElement)) {
+          focusedElement.blur();
+        }
+      }
+    } else if (node.nodeType === Node.TEXT_NODE && node.data !== "") {
+      // No CSS reaches a text node, and it stays where it is: Preact inserts
+      // next to it. Emptied for good, since the page is taken down without
+      // being shown again, and Preact only writes it when its text changes.
+      node.data = "";
+      observer.observe(node, {
+        characterData: true
+      });
     }
     node = node.nextSibling;
   }
@@ -50082,7 +50092,7 @@ const useControlProps = (props, {
     }
     const domProps = toDomProps(newUIState);
     Object.assign(el, domProps);
-    debugUIState(e, `syncDomState: updated to ${getElementSignature(el)}`, domProps);
+    debugUIState(e, `syncDomState: updated to ${getElementSignature(el)}`);
   };
   const controlInfo = createControlInfo(props, {
     controlType
@@ -82932,7 +82942,8 @@ const SelectableInput = SelectableInputProxy;
  * The `highlight` argument of `useSearchHighlight` (a row's `matchInfo.matchRanges`) can be:
  *   - an array of [start, end] pairs — applied to all text nodes under the root element
  *   - an object { [domSelector]: [[start, end], …] } — applied to each sub-element
- *     matched by the selector (the format produced by createSearch)
+ *     matched by the selector (the format produced by createSearch); a selector
+ *     matching nothing under the root element marks nothing
  */
 
 // Module-level shared Highlight instance — created lazily once.
@@ -82978,10 +82989,12 @@ const applySearchHighlight = (el, highlight) => {
     ? highlight.length === 0
       ? []
       : [{ root: el, ranges: highlight }]
-    : Object.entries(highlight).map(([selector, ranges]) => ({
-        root: el.querySelector(selector) ?? el,
-        ranges,
-      }));
+    : Object.entries(highlight).flatMap(([selector, ranges]) => {
+        // The ranges are offsets into the selected element's text: an item
+        // that doesn't render it has nothing to mark.
+        const root = el.querySelector(selector);
+        return root ? [{ root, ranges }] : [];
+      });
 
   if (entries.length === 0) {
     return undefined;
@@ -96728,7 +96741,12 @@ const mergeRanges = (ranges) => {
  *
  * Each field config:
  *   - getter(item): string — extracts the text to search
- *   - domSelector: string — CSS selector used by ListItem to find the target element
+ *   - domSelector?: string — CSS selector used by ListItem to find the target element.
+ *                   Without it, the field is searched and scored but highlights
+ *                   nothing: text the item doesn't write (an alias, a civil name
+ *                   under a nickname, tags). One field per selector: the ranges
+ *                   of two fields sharing one are offsets into two different
+ *                   strings, so the last field wins.
  *   - priority?: number — multiplier applied to the field's score (default 1)
  *   - matchFn?: function — custom match function (searchText, fieldValue) => { match, matchScore, matchRanges }
  *                          defaults to applySearch
@@ -96748,7 +96766,9 @@ const createSearch = (fields) => {
       const result = matchFn(searchText, fieldValue);
       if (result.match && result.matchRanges.length > 0) {
         totalScore += result.matchScore * priority;
-        matchRanges[domSelector] = result.matchRanges;
+        if (domSelector) {
+          matchRanges[domSelector] = result.matchRanges;
+        }
       }
     }
     if (totalScore === 0) {