@jsenv/navi 0.29.515 → 0.29.516

package/dist/dev/jsenv_navi.js

@@ -489,32 +489,33 @@ const MountAfterPaint = ({
 /**
  * Builds, now, every box still waiting whose top is above `bottom` (in the
  * document's coordinates): what the document will show once scrolled so that
- * `bottom` is the bottom of the screen. Boxes built push those below them
- * further down, so the ones measured above `bottom` are all the ones that can
- * be shown there, and a few more at worst.
+ * `bottom` is the bottom of the screen.
+ *
+ * Each box is read once those before it have their height. A box has no height
+ * until it is built, so read together they would all stand at the end of what
+ * is built, above `bottom` as soon as the offset is worth a screen: a page of
+ * sections would be built whole before its first frame. A box found below
+ * `bottom` waits: a build only pushes it further down, and where it stands
+ * already makes the document tall enough for the offset.
+ *
+ * The walk follows the set live, in mount order: a box mounted by a build is
+ * read too. One standing higher than boxes built before it (mounted later) can
+ * leave a few built that the offset does not show, never one missing that it
+ * does.
  *
  * Renders synchronously, so it must be called outside a Preact commit: a render
  * flushed from a layout effect runs inside the commit still walking the tree.
  */
 const buildDeferredBoxesAbove = bottom => {
-  if (deferredBoxes.size === 0) {
-    return;
-  }
-  const toBuild = [];
   for (const deferredBox of deferredBoxes) {
     const top = readTopInWindow(deferredBox.getElement());
-    if (top !== null && top + window.scrollY < bottom) {
-      toBuild.push(deferredBox);
+    if (top === null || top + window.scrollY >= bottom) {
+      continue;
     }
-  }
-  if (toBuild.length === 0) {
-    return;
-  }
-  flushSyncRendering(() => {
-    for (const deferredBox of toBuild) {
+    flushSyncRendering(() => {
       deferredBox.build();
-    }
-  });
+    });
+  }
 };
 
 // null for a box nothing draws (a closed popup, a hidden tab): no frame shows
@@ -10839,6 +10840,12 @@ const startAtTop = (url, { from } = {}) => {
   // position to come back to — the return would keep the offset of the page
   // being left.
   positionByUrl.set(href, { x: 0, y: 0 });
+  // A list holding the document on the page arriving has placed it already: a
+  // navigation holding the rendering gets here after that render, and nothing
+  // owes the list another commit to put it back (see holdDocumentScroll).
+  if (isDocumentHeld()) {
+    return;
+  }
   window.scrollTo({ top: 0, left: 0, behavior: "instant" });
 };
 // Also what decides whether a push carries the state of the page it keeps
@@ -14023,10 +14030,17 @@ const isRenderingPageChange = () => renderingPageChange;
  *   outside any render — by a subscription deciding whether to re-render, by
  *   the scroll restoration when a position is due — and it reads the routes
  *   untracked, so asking subscribes nobody to them.
- * - `leavingSignal` — true once the page is hidden and kept while its movement
- *   plays: still mounted, so what it fills or moves outside its own nodes (a
- *   slot in a bar, the document a list scrolls) steps out rather than showing
- *   with, or moving, the page arriving. Every box of it measures 0×0 by then.
+ * - `hasLeft()` — whether the page, or a page it sits in, is kept while leaving.
+ *   Written by the container's render, before anything below it renders, and
+ *   final. `isShown()` can answer false for a page that stays on screen (the
+ *   address coming back, a container first rendered while the routes are
+ *   frozen), so what cannot be undone asks this one.
+ * - `leavingSignal` — true once the page, or a page it sits in, is hidden and
+ *   kept while the movement plays: still mounted, so what it fills or moves
+ *   outside its own nodes (a slot in a bar, the document a list scrolls) steps
+ *   out rather than showing with, or moving, the page arriving. Every box of it
+ *   measures 0×0 by then. Set after the render keeping the page, so that render
+ *   and what it builds can only ask `hasLeft()`.
  */
 const RoutePageContext = createContext(null);
 
@@ -37886,7 +37900,7 @@ const useContentKeepingLeavingPages = (branch, readShownBranch, parentPage) => {
         continue;
       }
       entry.hidden = true;
-      entry.page.leavingSignal.value = true;
+      entry.page.ownLeavingSignal.value = true;
       // 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, a text it writes shows
@@ -37974,11 +37988,21 @@ const hideNodesBetween = (start, end, observer) => {
 // page it sits in is shown itself. Never again once kept while leaving: coming
 // back to its route mounts the page anew.
 const createRoutePage = (readShownBranch, parentPage) => {
+  const ownLeavingSignal = signal(false);
   const page = {
     branchId: null,
     element: undefined,
     left: false,
-    leavingSignal: signal(false),
+    ownLeavingSignal,
+    // A page nested in one that left is hidden with it, even while its own
+    // container goes on picking it (a route matching on both sides).
+    leavingSignal: parentPage ? computed(() => ownLeavingSignal.value || parentPage.leavingSignal.value) : ownLeavingSignal,
+    hasLeft: () => {
+      if (page.left) {
+        return true;
+      }
+      return parentPage ? parentPage.hasLeft() : false;
+    },
     isShown: () => {
       if (page.left) {
         return false;
@@ -87454,27 +87478,35 @@ const useListScrollSync = ({
   // landing or resizing — and said again when it reads differently (`ifChanged`):
   // the same scroll, the screen said right.
   const announcedRef = useRef(null);
+  // A scroll landing past everything drawn (a key sending the list to its end,
+  // a hand on the scrollbar) has no item to read a position from: it is said by
+  // that re-read, even when it is the first thing the list has to say.
+  const reportOwedRef = useRef(false);
   const reportPosition = ({
     ifChanged
   } = {}) => {
     if (!ref.current) {
       return;
     }
-    if (ifChanged && !announcedRef.current) {
+    if (ifChanged && !announcedRef.current && !reportOwedRef.current) {
       return;
     }
     const remember = rememberScrollRef.current;
     const onScrolledChange = onScrolledChangeRef.current;
     const position = captureListAnchor(remember || Boolean(onScrolledChange));
     if (!position) {
+      if (!ifChanged) {
+        reportOwedRef.current = true;
+      }
       return;
     }
+    reportOwedRef.current = false;
     positionRef.current = position;
     if (!remember && !onScrolledChange) {
       return;
     }
     const scrolledNow = toScrolledPosition(position);
-    if (ifChanged && isSamePosition(scrolledNow, announcedRef.current)) {
+    if (ifChanged && announcedRef.current && isSamePosition(scrolledNow, announcedRef.current)) {
       return;
     }
     announcedRef.current = scrolledNow;
@@ -88030,10 +88062,9 @@ const useListScrollSync = ({
     // ends the hold like a wheel does — the ones that come with no input event
     // at all: the status bar tap on iOS, find in page, a script.
     // Only once the list has stood where it holds itself, though. Until then
-    // the page is still settling around it, and some of that is written after
-    // the list placed itself: a navigation holding the rendering for its
-    // transition takes the arriving page to its top once the page rendered
-    // (whenRenderingResumes), and the hold puts it back at its next commit.
+    // the page is still settling around it, and what that moves (a document
+    // still too short for the offset clamps it) the hold puts back at its next
+    // commit.
     const letGoUnlessHeldScroll = () => {
       const place = startPlaceRef.current;
       const way = place.way;
@@ -109061,9 +109092,9 @@ SidePanel.Foot = SidePanelFoot;
  * mount, or the first to render into an empty slot; later renders of a
  * `SlotFill` that is not the filler leave the slot alone. A `SlotFill`
  * unmounting empties the slot only while it is still that one, and so does one
- * in a page a route transition keeps while leaving (see keepLeavingPages in
- * nav/route.jsx): still mounted, hidden, it must neither stay in the slot nor
- * take it back from the page arriving.
+ * in or below a page a route transition keeps while leaving (see
+ * keepLeavingPages in nav/route.jsx): still mounted, hidden, it must neither
+ * stay in the slot nor take it back from the page arriving.
  *
  * **One `SlotFill`, rendered where the choice is made.** `isFilled` follows
  * the fills as they mount and unmount, in the order the tree walks them. A
@@ -109128,8 +109159,10 @@ const createSlot = (SlotRenderer = Box) => {
     }, []);
     // In a page kept while leaving: out of the slot as if unmounted, since
     // the page arriving is what the slot must show — or nothing, when it
-    // fills none.
-    if (routePage && routePage.leavingSignal.value) {
+    // fills none. The signal brings the render that steps out; hasLeft() refuses
+    // a fill built in a kept page (a wrapper changing shape rebuilds what it
+    // holds), which renders before that signal is set.
+    if (routePage && (routePage.leavingSignal.value || routePage.hasLeft())) {
       if (filler === fillerRef) {
         filler = null;
         slotPropsSignal.value = null;