@jsenv/navi 0.27.58 → 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}`,
@@ -35663,6 +35672,9 @@ const Dialog = props => {
       };
       closeRequestHandler(e, closePermission, detail);
       if (denied) {
+        if (e.type === "cancel") {
+          e.preventDefault();
+        }
         closePermission.allow = () => {
           close(e, detail);
         };
@@ -35713,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 => {