Changeset e579a98 in Klonkt

Timestamp:

06/21/2026 05:04:21 PM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

097da22

Parents:

2520dcc

Message:

fix(zoeken): bottom-tab search button stopped working after htmx navigation

The button captured #search-overlay once, but that overlay lives in the chrome
which is OOB-replaced on every htmx nav → reference went stale → button did
nothing. Now via delegation + looking up the overlay fresh on each click (as the
desktop button already did). Guard against double-wiring.

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

File:

• src/views/partials/bottom-tab.ejs (modified) (1 diff)

src/views/partials/bottom-tab.ejs

@@ -321 +321 @@
 
 <script>
-// Wire the bottom-tab search button into the existing #search-overlay
+// Bottom-tab zoekknop → opent #search-overlay. Via event-delegatie + de overlay
+// ELKE klik opnieuw opzoeken: de overlay zit in de chrome die bij htmx-navigatie
+// OOB vervangen wordt, dus een één-keer-opgevangen referentie verschaalt
+// (knop deed daarna niets). Guard tegen dubbel-wiren.
 (function() {
-  const btn = document.getElementById('bottom-tab-search-toggle');
-  const overlay = document.getElementById('search-overlay');
-  if (btn && overlay) {
-    btn.addEventListener('click', function() {
-      overlay.hidden = false;
-      const inp = overlay.querySelector('input');
-      if (inp) inp.focus();
-    });
-  }
+  if (window.__btSearchWired) return;
+  window.__btSearchWired = true;
+  document.addEventListener('click', function(e) {
+    if (!e.target.closest || !e.target.closest('#bottom-tab-search-toggle')) return;
+    var o = document.getElementById('search-overlay');
+    if (o) { o.hidden = false; var i = o.querySelector('input'); if (i) i.focus(); }
+  });
 })();
 </script>