Changeset cf0de0c in Klonkt

Timestamp:

06/20/2026 08:21:47 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

d8c6a83

Parents:

1c6d61c

Message:

fix(nav): syncAria also on pcmsNav + delayed retry (htmx race)

syncAria sometimes ran on htmx:afterSettle before the body class update →
Timeline/Grid stayed incorrectly active after an htmx click (e.g. to the
calendar). Now syncAria also fires on pcmsNav (fires after the class update)
+ 60ms retry to handle OOB-swap races.

Co-Authored-By: Claude <noreply@…>

File:

• src/views/shell.ejs (modified) (1 diff)

src/views/shell.ejs

@@ -397 +397 @@
   }
   syncAria();
-  // Re-sync after HTMX brings in new content (e.g. navigating back to home)
-  document.body.addEventListener('htmx:afterSettle', syncAria);
+  // Re-sync after HTMX brings in new content (e.g. navigating back to home).
+  // pcmsNav vuurt ná de body-class-update (zie de pcmsNav-listener hierboven), dus
+  // dáár weet syncAria de juiste pagina-class — los van de afterSettle-timing.
+  // Een vertraagde herhaling wint eventuele resterende races (OOB-chrome-swap).
+  document.body.addEventListener('htmx:afterSettle', function(){ syncAria(); setTimeout(syncAria, 60); });
+  document.body.addEventListener('pcmsNav', function(){ syncAria(); setTimeout(syncAria, 60); });
 
   // Pages where the body actually has a feed to toggle. On these the