Changeset 25dd08f in Klonkt for src/assets/js/mod/playlist-editor.js

Timestamp:

08/22/2026 08:24:13 AM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

8e1af9c

Parents:

b0ccd0d

Message:

Mixtape kiezen kan nu ook echt, in het scherm

De soort was er wel maar nergens te kiezen. Drie plekken, en twee ervan waren
weer dezelfde vorm die een derde soort niet afwijst maar opslokt:

• De keuzelijst kreeg een derde optie.
• kind: kindEl.value === 'playlist' ? 'playlist' : 'album' sloeg een mixtape op als album. Nu gaat de waarde zelf mee; normKind() op de server beslist.
• const album = kindEl.value !== 'playlist' toonde uitgavevelden bij een mixtape, die de server vervolgens leegmaakt -- een scherm dat iets anders belooft dan het doet. Nu expliciet === 'album'.
• De lijstweergave koos teken en woord met een ternair die er twee kende; een mixtape kreeg daardoor het albumjasje. Nu een tabel per soort.

Labels in nl/en/de, en de sleutel doorgegeven vanuit de partial -- dat laatste
ving playlist-editor-i18n.test.js meteen, precies waarvoor die test bestaat.

MOD_V naar 50.

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

File:

• src/assets/js/mod/playlist-editor.js (modified) (3 diffs)

src/assets/js/mod/playlist-editor.js

@@ -123 +123 @@
                     <option value="album"    ${initial.kind === 'album' ? 'selected' : ''}>💿 ${T('k_album')}</option>
                     <option value="playlist" ${initial.kind === 'playlist' ? 'selected' : ''}>📃 ${T('k_playlist')}</option>
+                    <option value="mixtape"  ${initial.kind === 'mixtape' ? 'selected' : ''}>📼 ${T('k_mixtape')}</option>
                   </select>
                 </label>
@@ -197 +198 @@
     // verdwijnen dus als je omschakelt, en de server maakt ze dan ook leeg;
     // dit scherm is de uitleg, niet de bewaking.
+    // ALLEEN een album, en niet "alles behalve playlist". Dat stond er eerst, en
+    // met een derde soort erbij zou een mixtape ineens uitgavevelden tonen die
+    // de server toch weggooit -- een scherm dat iets anders belooft dan het doet.
     const toonUitgave = () => {
-      const album = kindEl.value !== 'playlist';
+      const album = kindEl.value === 'album';
       for (const el of backdrop.querySelectorAll('.pl-uitgave')) el.hidden = !album;
     };
@@ -420 +424 @@
         year:   parseInt(yearEl.value, 10) || 0,
         cover:  coverEl.value.trim(),
-        kind:   kindEl.value === 'playlist' ? 'playlist' : 'album',
+        // De waarde zelf. Deze regel was `=== 'playlist' ? 'playlist' : 'album'`,
+        // en dat is precies de vorm die een nieuwe soort niet afwijst maar
+        // opslokt: een mixtape werd hier stil een album. De server heeft het
+        // laatste woord via normKind().
+        kind:   kindEl.value,
         release_date:  releaseEl.value.trim(),
         mb_release_id: mbReleaseEl.value.trim(),