@jsenv/navi 0.29.326 → 0.29.328

package/dist/dev/jsenv_navi.js

@@ -7325,6 +7325,16 @@ const positionCallout = (
           }
           return;
         }
+        if (!calloutElement.isConnected) {
+          // The callout is drawn inside the anchor's subtree (calloutContainer),
+          // so it leaves the document with it — a Suspense boundary parking
+          // the screen while an action re-runs keeps the subtree mounted, refs
+          // intact, but detached. showPopover() throws on a detached element,
+          // and leaving the document already took it out of the top layer, so
+          // there is nothing to hide either: same as ancestorClosed, wait for
+          // the next check, which shows it again once the subtree is back.
+          return;
+        }
         if (!calloutElement.matches(":popover-open")) {
           if (debug) {
             debug(event, "showing callout because anchor is visible again");
@@ -11288,9 +11298,11 @@ const swipeTypeOf = (axis, pulled) => {
  * copy is captured where it lands rather than where it was let go of.
  *
  * And the promise matters in both cases: the gesture holds its copy until the
- * answer settles. Returning the transition is what makes a landing continuous; a
- * `toss` that rejects brings the copy back, because the thing still exists and the
- * screen has to say so.
+ * answer settles. Returning the transition is what makes a landing continuous;
+ * rejecting brings the copy back over the original, because the thing still
+ * exists and the screen has to say so. That holds for every outcome that carries
+ * a copy — `land` and `reorder` as much as `toss` and `leave` — so a place can be
+ * asked about rather than taken: refuse, and the board is as it was.
  *
  * Starting a document transition is the application's call and not navi's: a
  * `view-transition-name` must be unique per document, so only the application can
@@ -27186,20 +27198,28 @@ const setupBrowserIntegrationViaHistory = ({
   // traversal to another document is a full page load, which no press on a
   // link asked for — the entries that are ours are recorded as they are
   // created, starting with the one this document was loaded into.
+  //
+  // The presence of window.navigation says nothing about the entry: the spec
+  // makes currentEntry null in a document that is not fully active, and iOS
+  // Safari answers null in a document that plainly is (a booted app, minutes
+  // into a session). Nothing can be done about it; an entry that cannot be
+  // read is treated as a browser without the API, which only costs the
+  // adjacency.
   const sameDocumentEntryKeys = new Set();
   const rememberEntryIsOfThisDocument = () => {
-    if (window.navigation) {
-      sameDocumentEntryKeys.add(window.navigation.currentEntry.key);
+    const currentEntry = window.navigation?.currentEntry;
+    if (currentEntry) {
+      sameDocumentEntryKeys.add(currentEntry.key);
     }
   };
   rememberEntryIsOfThisDocument();
   const adjacentEntryDelta = (url) => {
-    const { navigation } = window;
-    if (!navigation) {
+    const currentEntry = window.navigation?.currentEntry;
+    if (!currentEntry) {
       return 0;
     }
-    const entries = navigation.entries();
-    const index = navigation.currentEntry.index;
+    const entries = window.navigation.entries();
+    const index = currentEntry.index;
     // Behind first: when the same page stands on both sides (A, B, A and one
     // is on B), a link to it reads as going back.
     for (const delta of [-1, 1]) {