stx-router 0.2.248 → 0.2.250

package/dist/client.js

@@ -1329,6 +1329,21 @@ else {
   // mode where every page is part of the same SPA shell.
   document.addEventListener('click',function(e){
     if(e.metaKey||e.ctrlKey||e.shiftKey||e.altKey||e.button!==0)return;
+    // A handler on the page has already decided this click is not a
+    // navigation, so the router must not perform one - the same contract the
+    // browser honours for a plain anchor.
+    //
+    // This listener used to run in the CAPTURE phase, which broke that twice
+    // over: it saw the click before the anchor did, so defaultPrevented was
+    // necessarily false by the time it was read here, and the stopPropagation
+    // below then stopped the event ever reaching the element - so a page's own
+    // click handler on an intercepted link never ran AT ALL. Progressive
+    // enhancement on an anchor was impossible; the only way out was to swap it
+    // for a button in script (stacksjs/stacks#2393).
+    //
+    // Bubbling instead puts the page's handlers first, which is where a
+    // decision about the page's own markup belongs.
+    if(e.defaultPrevented)return;
     if(!e.target||!e.target.closest)return;
     var link=e.target.closest('[data-stx-link]');
     // A marked link still has to clear the opt-out set. It used to skip it
@@ -1346,7 +1361,7 @@ else {
     e.stopPropagation();
     log('[router] navigating to:',href);
     navigate(withCurrentLocale(href));
-  },true);
+  });
 
   // \u2500\u2500 Form submission \u2500\u2500
   // A submit IS a navigation, and the router only ever watched clicks. So any

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "stx-router",
   "type": "module",
-  "version": "0.2.248",
+  "version": "0.2.250",
   "description": "File-based router for STX — .stx template discovery, nested layouts, typed route params, middleware, and client-side SPA navigation.",
   "author": "Chris Breuer <chris@stacksjs.org>",
   "license": "MIT",