@jsenv/navi 0.29.327 → 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");
@@ -27188,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]) {