@jsenv/navi 0.27.57 → 0.27.59

package/dist/jsenv_navi.js

@@ -14285,9 +14285,14 @@ This prevents cross-test pollution and ensures clean state.`,
               // 1. We're navigating within the same route family (not to completely unrelated routes)
               // 2. AND no matching route extracts this parameter from URL
               // 3. AND parameter has no default value (making it truly optional)
+              // 4. AND parameter is a path segment (not a search param)
+              //    Search params represent user preferences/choices and must survive
+              //    navigation away — only path segments are explicitly removed from
+              //    the URL when the route stops matching.
               if (
                 matchingRouteInSameFamily &&
-                !parameterExtractedByMatchingRoute
+                !parameterExtractedByMatchingRoute &&
+                pathConnectionMap.has(paramName)
               ) {
                 const defaultValue = connection.getDefaultValue();
                 if (defaultValue === undefined) {
@@ -14309,6 +14314,10 @@ This prevents cross-test pollution and ensures clean state.`,
                   console.debug(
                     `[route] Different route family: preserving ${paramName} signal value: ${paramSignal.value}`,
                   );
+                } else if (queryConnectionMap.has(paramName)) {
+                  console.debug(
+                    `[route] Search param ${paramName}: preserving signal value (user choice): ${paramSignal.value}`,
+                  );
                 } else {
                   console.debug(
                     `[route] Parameter ${paramName} extracted by matching route: preserving signal value: ${paramSignal.value}`,
@@ -14718,8 +14727,15 @@ const setupBrowserIntegrationViaHistory = ({
       updateDocumentState(state);
     }
 
-    // Routes only match on URL — skip route matching for state-only changes.
-    if (isSameUrl && navigationType !== "reload" && navigationType !== "load") {
+    // Skip route matching for state-only changes: push/replace to the same URL
+    // (e.g. useNavState updating document state without changing the route).
+    // Do NOT apply for "traverse" — window.location.href is already updated by
+    // the browser before the popstate handler runs, so isSameUrl is always true
+    // for back/forward navigation regardless of whether the URL actually changed.
+    if (
+      isSameUrl &&
+      (navigationType === "push" || navigationType === "replace")
+    ) {
       return undefined;
     }
 
@@ -35656,6 +35672,9 @@ const Dialog = props => {
       };
       closeRequestHandler(e, closePermission, detail);
       if (denied) {
+        if (e.type === "cancel") {
+          e.preventDefault();
+        }
         closePermission.allow = () => {
           close(e, detail);
         };
@@ -35706,9 +35725,6 @@ const Dialog = props => {
       }
     },
     onCancel: e => {
-      // The browser fires "cancel" (then closes the dialog) when the user presses Escape.
-      // Prevent the native close so we control the close flow and dispatch navi_dialog_close.
-      e.preventDefault();
       onRequestClose(e);
     },
     onnavi_request_open: e => {