@jsenv/navi 0.30.36 → 0.30.38

package/dist/dev/jsenv_navi.js

@@ -30518,8 +30518,9 @@ const SCOPED_MANY_OPTIONS = new Set([
  *   policy as that verb, and is a write of its own for a dependent:
  *   `{ resource: ORDER, on: ["cancel"] }`. Its `rerunOn` says which reads of
  *   this resource it reruns (`{ GET_RANGE: true }`); a read it leaves out
- *   follows `rerunOn` for its verb. A name a verb takes, or one the resource
- *   already exposes (`withParams`, `store`…), throws. A `.scopedOne()` or
+ *   follows `rerunOn` for its verb. A verb's name in any case (`post`,
+ *   `get_many`…), or one the resource already exposes (`withParams`, `store`…),
+ *   throws. A `.scopedOne()` or
  *   `.scopedMany()` takes `writes` too, `.scopedMany()` with the plural verbs.
  * @param {{ GET?: { ms?: number, max?: number }, GET_MANY?: { ms?: number, max?: number } }} [restCallbacks.keep] -
  *   how long an answer of a read is trusted, coming back to the same params
@@ -32084,12 +32085,15 @@ ${name}.many("${propertyName}") source location: ${declarationSite}`,
 
   // expose one action per write named in `writes`
   if (writes) {
+    const membersExposed = Object.keys(stateFacade).filter(
+      (key) => !VERB_KEYS.includes(key),
+    );
     for (const [writeName, write] of Object.entries(writes)) {
       assertWriteCanBeNamed(
         name,
         writeName,
         write,
-        stateFacade,
+        membersExposed,
         namedWriteVerbs,
       );
       const { verb: writeVerb, callback, rerunOn: writeRerunOn } = write;
@@ -32402,7 +32406,7 @@ const assertWriteCanBeNamed = (
   name,
   writeName,
   write,
-  stateFacade,
+  membersExposed,
   namedWriteVerbs,
 ) => {
   const where = `resource(${name}) writes.${writeName}`;
@@ -32411,9 +32415,9 @@ const assertWriteCanBeNamed = (
       `${where}: "${writeName}" is a verb's name — a write named in "writes" takes a name of its own (cancel, archive…)`,
     );
   }
-  if (Object.hasOwn(stateFacade, writeName)) {
+  if (membersExposed.includes(writeName)) {
     throw new Error(
-      `${where}: the resource already exposes "${writeName}" — pick another name`,
+      `${where}: the resource already exposes "${writeName}" — a write takes a name none of its members has: ${membersExposed.join(", ")}`,
     );
   }
   if (!write || typeof write !== "object") {
@@ -42628,7 +42632,12 @@ const watchRowScroll = (
 };
 
 installImportMetaCssBuild(import.meta);
-const css$16 = /* css */`.navi_route_travel {
+const css$16 = /* css */`@property --navi-route-row-pin {
+  syntax: "*";
+  inherits: false
+}
+
+.navi_route_travel {
   flex-direction: column;
   display: flex;
   position: relative;
@@ -42684,7 +42693,7 @@ const css$16 = /* css */`.navi_route_travel {
     display: flex;
   }
 
-  & > [data-route-row-pane][inert]:not([data-route-row-held]), &:has( > [data-route-row-held]) > [data-route-row-pane]:not([data-route-row-held]) {
+  & > [data-route-row-pane][inert]:not([data-route-row-held]), &[data-route-row-holding] > [data-route-row-pane]:not([data-route-row-held]) {
     top: var(--navi-route-row-pin, 0);
     contain: paint;
     max-height: 100dvh;
@@ -42856,11 +42865,25 @@ const RouteTravel = ({
   // when the row rests. Not what readers are told the page held is
   // (route_travel_hold.js): that follows the hand, and goes at its lift.
   const heldPaneRef = useRef(null);
-  const releasePaneAtRest = () => {
+  const releasePaneAtRest = row => {
     const heldPane = heldPaneRef.current;
     if (heldPane) {
       heldPaneRef.current = null;
       heldPane.removeAttribute("data-route-row-held");
+      row.removeAttribute("data-route-row-holding");
+    }
+  };
+
+  // How far down the reader is looking, on every pane, for those the CSS pins.
+  const pinPanes = row => {
+    const pin = `${pinOf(row)}px`;
+    for (const pane of panesOf(row)) {
+      pane.style.setProperty("--navi-route-row-pin", pin);
+    }
+  };
+  const unpinPanes = row => {
+    for (const pane of panesOf(row)) {
+      pane.style.removeProperty("--navi-route-row-pin");
     }
   };
 
@@ -42887,13 +42910,14 @@ const RouteTravel = ({
     if (!windowed && paneAtRest) {
       heldPaneRef.current = paneAtRest;
       paneAtRest.setAttribute("data-route-row-held", "");
+      row.setAttribute("data-route-row-holding", "");
     }
     if (movement.scrollsDocument) {
       // Whatever the browser does with the offset meanwhile (clamping it to a page
       // shorter than the trip) is not the reader scrolling, and the address it
       // would be written against may already be another page's.
       movement.resumeScrollRecording = suspendScrollRecording();
-      row.style.setProperty("--navi-route-row-pin", `${pinOf(row)}px`);
+      pinPanes(row);
     }
     return movement;
   };
@@ -43186,7 +43210,7 @@ const RouteTravel = ({
     row.removeAttribute("data-route-row-taken");
     publishRouteRowMove(null);
     releaseHeldPage();
-    releasePaneAtRest();
+    releasePaneAtRest(row);
     if (liveRef.current) {
       liveRef.current.style.top = "";
     }
@@ -43194,7 +43218,7 @@ const RouteTravel = ({
       picture.remove();
     }
     if (movement.scrollsDocument) {
-      row.style.removeProperty("--navi-route-row-pin");
+      unpinPanes(row);
       const {
         pages,
         currentIndex
@@ -43499,7 +43523,7 @@ const RouteTravel = ({
       movementRef.current = null;
       publishRouteRowMove(null);
       releaseHeldPage();
-      releasePaneAtRest();
+      releasePaneAtRest(row);
     };
   }, [windowed]);
 
@@ -43563,7 +43587,7 @@ const RouteTravel = ({
     const row = rowRef.current;
     const onTouchStart = () => {
       if (!movementRef.current && rowFollowsDocument(row)) {
-        row.style.setProperty("--navi-route-row-pin", `${pinOf(row)}px`);
+        pinPanes(row);
       }
     };
     row.addEventListener("touchstart", onTouchStart, {
@@ -84367,9 +84391,15 @@ const PickerNaviMinute = props => {
  * element crossing the cursor — a dozen per wheel tick. Those hovers are noise:
  * the user asked to scroll, not to hover. Anything doing real work on hover (a
  * map highlight, a preview, a prefetch) then pays for that noise on the main
- * thread, exactly while a scroll animation is running. A rule matching
- * `[navi-scrolling] *` takes the rows out of hit-testing and the browser
- * suppresses it all, at no cost per element.
+ * thread, exactly while a scroll animation is running. A rule like
+ * `[navi-scrolling] .my_row { pointer-events: none }` takes the rows out of
+ * hit-testing and the browser suppresses it all.
+ *
+ * Such a rule must end on a class (or an id, a tag, an attribute) of what it
+ * styles: Chrome then restyles only those elements when the attribute comes and
+ * goes. One ending on `*`, including `.my_row:is([navi-scrolling] *)`, gives it
+ * nothing to narrow to, so it restyles everything under the scroller (the whole
+ * document when the page scrolls) at the start and at the end of every scroll.
  *
  * One capturing listener on the document sees them all: "scroll" does not
  * bubble, but it does propagate in the capture phase.
@@ -86754,7 +86784,7 @@ const css$w = /* css */`@property --list-group-label-height {
     }
   }
 
-  &:not([navi-hover-while-scrolling]) > .navi_list_scroll_container > .navi_list:is([navi-scrolling] *) {
+  [navi-scrolling] &:not([navi-hover-while-scrolling]) > .navi_list_scroll_container > .navi_list, &[navi-scrolling]:not([navi-hover-while-scrolling]) > .navi_list_scroll_container > .navi_list, &:not([navi-hover-while-scrolling]) > .navi_list_scroll_container[navi-scrolling] > .navi_list {
     pointer-events: none;
   }
 
@@ -92780,8 +92810,19 @@ const useItemStore = ({
   // was asked for, and the window keeps growing meanwhile: dropping what the
   // page did not cover empties items on screen.
   const staleRanksRef = useRef(null);
+  // A run standing for no item has no rank to mark: its count is what is from
+  // before, and no window frames anything that would ask for it again.
+  const countStaleRef = useRef(false);
+  const markHeldStale = () => {
+    staleRanksRef.current = new Set(pages.byIndex.keys());
+    countStaleRef.current = pages.count === 0;
+  };
   if (staleRanksRef.current === null) {
-    staleRanksRef.current = restored ? new Set(pages.byIndex.keys()) : new Set();
+    if (restored) {
+      markHeldStale();
+    } else {
+      staleRanksRef.current = new Set();
+    }
   }
   const takeInPage = ({
     pageItems,
@@ -92820,6 +92861,7 @@ const useItemStore = ({
       i++;
     }
     pages.count = pageCount;
+    countStaleRef.current = false;
     // Which rank holds which id, kept by the source so a list drawing
     // this collection again finds it drawn (see readComposition above).
     if (itemsAction.writeComposition) {
@@ -92855,7 +92897,7 @@ const useItemStore = ({
   const invalidationRef = useRef(invalidation);
   if (invalidationRef.current !== invalidation) {
     invalidationRef.current = invalidation;
-    staleRanksRef.current = new Set(pages.byIndex.keys());
+    markHeldStale();
   }
   // The one request in flight, with the means to call it off: a page asked for
   // a window the list has left is work the server and the browser are doing for
@@ -93053,7 +93095,8 @@ const useItemStore = ({
           rank++;
         }
       }
-      const revalidating = staleFrom !== -1;
+      const countStale = countStaleRef.current;
+      const revalidating = staleFrom !== -1 || countStale;
       // The list is held on an item nothing on screen leads to: the items it holds
       // do not contain it, so no window it could draw will ever bring it. Only
       // asking for it by name does.
@@ -93069,6 +93112,10 @@ const useItemStore = ({
         const from = typeof wanted.index === "number" ? rankOfIndex(wanted.index) - Math.floor(budget / 2) : 0;
         start = from < 0 ? 0 : from;
         end = start + budget - 1;
+      } else if (countStale) {
+        // Nothing drawn to anchor on: the first page, like the very first ask.
+        start = 0;
+        end = budget - 1;
       } else if (revalidating) {
         // The items from before that the window draws, and the missing ones
         // with them: one ask.
@@ -93270,6 +93317,7 @@ const useItemStore = ({
               staleRanksRef.current.delete(rank);
               rank++;
             }
+            countStaleRef.current = false;
             setRefreshing(false);
             return;
           }