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

Timestamp:

08/09/2026 07:47:48 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

b406d25

Parents:

c832064

Message:

Audio is geen keuze meer: Album en Playlist staan in de balk (shaer-cyg)

De posttype-balk had post/foto/video/audio. Audio verdwijnt als keuze -- alles
wat muziek is landt voortaan op album of playlist, en dat maakt de keuze
betekenisvol in plaats van cosmetisch. Album en Playlist delen het muziekpaneel,
want het verschil zit in de playlist en niet in wat je uploadt.

De knop Audio blijft alleen staan zolang een post nog type=audio IS. Anders zou
het openen van een oude post hem stilzwijgend van type veranderen, en dat is
precies het soort verlies dat niemand ziet gebeuren.

De lijst van geldige types stond op DRIE plaatsen los van elkaar: twee keer in
routes/posts.js en een keer in routes/types.js. Dat viel niet op zolang ze
gelijk waren, maar het faalt stil: kent het opslaan 'playlist' niet, dan wordt
de post zonder melding een gewone post en is de keuze verdwenen in plaats van
geweigerd. Nu een lijst, in config/post-types.js, die ook de editor voedt.

En de migratie: scripts/backfill-post-types.mjs zet bestaande audio-posts om via
postMusicType. Proefdraai is de standaard, --doen schrijft. Wat de afleiding
niet tot album of playlist maakt blijft met rust en wordt apart gemeld -- die
verdienen een blik en geen gok.

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

File:

• src/assets/js/mod/post-edit.js (modified) (1 diff)

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

@@ -980 +980 @@
         b.setAttribute('aria-checked', on ? 'true' : 'false');
       });
-      panels.forEach(p => { p.hidden = (p.dataset.panel !== tt); });
+      // data-panel mag meerdere types noemen: Album en Playlist delen het
+      // muziekpaneel, want het verschil zit in de playlist en niet in de upload.
+      panels.forEach(p => {
+        const voor = String(p.dataset.panel || '').trim().split(/\s+/);
+        p.hidden = !voor.includes(tt);
+      });
     }
     seg.addEventListener('click', (e) => {