Changeset 4b5db37 in Klonkt for src/assets/js/mod/post-edit.js

Timestamp:

08/07/2026 06:33:54 PM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

da84110

Parents:

c1df3e9

git-author:

Robin <roboburr@…> (08/07/2026 06:33:41 PM)

git-committer:

Robin <roboburr@…> (08/07/2026 06:33:54 PM)

Message:

Modules herstarten per paginawissel: de editor leeft weer na een terugkeer (shaer-5s1)

Drie klachten, een wortel. De oude inline scripts draaiden bij ELKE render;
een ES-module draait zijn top-level EEN keer per sessie, en de bootstrap
importeert een geladen module nooit opnieuw. Dus:

• /posts/new na een htmx-terugkeer: geen toolbar, geen serialisatie -- het verborgen veld kreeg de chips nooit terug (shortcodes 'opgegeten')
• /admin/audio na een terugkeer: de eigen kopieerhandler dood, en de klik viel door naar de GEDELEGEERDE handler van admin-media, die voor mediapaden terecht location.origin voorplakt -- vandaar ​https://site[[track:uuid]] op het klembord

De afspraak is nu: een module die per render moet draaien exporteert init(),
en de bootstrap roept die aan bij elke wissel waarop de module actief is.
Modules zonder init houden hun oude gedrag. Listeners op document/window
overleven de swap met closures naar dode elementen; makeSweeper in lib.js
veegt bij elke init de vorige lichting weg. Gedelegeerde handlers die
bewust blijven leven (media/videos) krijgen een paginawacht, want data-copy
betekent daar een PAD en elders een shortcode.

Omgezet: post-edit, admin-audio, admin-playlists. Bewaakt: admin-media,
admin-videos. playlist-editor en track-editor waren al swap-bestendig
(globale functie, gedelegeerd met eigen guards).

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

File:

• src/assets/js/mod/post-edit.js (modified) (5 diffs)

src/assets/js/mod/post-edit.js

@@ -5 +5 @@
 // het houdt een apostrof in een vertaling uit de HTML die hier geplakt wordt.
 
-import { pageData, esc } from './lib.js';
-
-const T = pageData();
+import { pageData, esc, makeSweeper } from './lib.js';
+
+// De oude inline scripts draaiden bij ELKE render; een module draait zijn
+// top-level een keer per sessie. Vandaar init(): de bootstrap roept hem aan
+// bij elke paginawissel waarop deze module actief is, en de veger haalt eerst
+// de document/window-listeners van de vorige pagina weg -- die overleven de
+// swap, met closures naar elementen die al verdwenen zijn (shaer-5s1).
+const doc = makeSweeper();
+let T = {};
+let _barObserver = null;
+
+export function init() {
+  doc.sweep();
+  if (_barObserver) { _barObserver.disconnect(); _barObserver = null; }
+  T = pageData();
+  run();
+}
+
+function run() {
 
           (function () {
@@ -631 +647 @@
   }
   function toggleFullscreen() { if (isFs()) closeFs(); else openFs(); }
-  window.addEventListener('popstate', function () { if (isFs()) applyFs(false); });
+  doc.on(window, 'popstate', function () { if (isFs()) applyFs(false); });
   if (fsBtn) fsBtn.addEventListener('click', toggleFullscreen);
   var fsDoneBtn = document.getElementById('pe-fs-done');
   if (fsDoneBtn) fsDoneBtn.addEventListener('click', closeFs);
-  document.addEventListener('keydown', (e) => {
+  doc.on(document, 'keydown', (e) => {
     if (e.key === 'Escape' && isFs()) { e.preventDefault(); closeFs(); }
   });
@@ -663 +679 @@
     if (bqBtn) bqBtn.classList.toggle('is-active', !!blockquoteAncestor());
   }
-  document.addEventListener('selectionchange', () => {
+  doc.on(document, 'selectionchange', () => {
     if (document.activeElement === editor) updateToolbarState();
   });
@@ -884 +900 @@
       }
     });
-    document.addEventListener('keydown', (e) => {
+    doc.on(document, 'keydown', (e) => {
       if (!trackPicker.hidden && e.key === 'Escape') {
         e.preventDefault();
@@ -1092 +1108 @@
   }
   position();
-  window.addEventListener('resize', position);
-  if (vv) { vv.addEventListener('resize', position); vv.addEventListener('scroll', position); }
-  // has-audio-player is toggled via a body class → observe it.
-  try { new MutationObserver(position).observe(document.body, { attributes: true, attributeFilter: ['class'] }); } catch (_) {}
+  doc.on(window, 'resize', position);
+  if (vv) { doc.on(vv, 'resize', position); doc.on(vv, 'scroll', position); }
+  // has-audio-player is toggled via a body class → observe it. De observer
+  // overleeft de swap net als de listeners; init() disconnect de vorige.
+  try { _barObserver = new MutationObserver(position); _barObserver.observe(document.body, { attributes: true, attributeFilter: ['class'] }); } catch (_) {}
 })();
+}