Changeset 25dd08f in Klonkt for src

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@…>

Location:

src

Files:

• assets/js/mod/playlist-editor.js (modified) (3 diffs)
• services/i18n.js (modified) (6 diffs)
• views/pages/admin-playlists.ejs (modified) (3 diffs)
• views/partials/playlist-editor.ejs (modified) (1 diff)
• views/shell.ejs (modified) (1 diff)

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(),

src/services/i18n.js

@@ -91 +91 @@
     'ple.type': 'Type',
     'ple.k_album': 'Album (genummerd)',
-    'ple.k_playlist': 'Playlist (track-covers)',
+    'ple.k_playlist': 'Playlist (track-covers)', 'ple.k_mixtape': 'Mixtape (bandje: alleen vooruit en achteruit)',
     'ple.uitgave': 'Uitgavedatum',
     'ple.mb_release': 'MusicBrainz release-id',
@@ -475 +475 @@
     'apl.none_sub': 'Maak er een via de knop hierboven, of via de 📃 knop in de post-editor.',
     'apl.pill_playlist': 'playlist',
-    'apl.pill_album': 'album',
+    'apl.pill_album': 'album', 'apl.pill_mixtape': 'mixtape',
     'apl.track': 'track',
     'apl.tracks': 'tracks',
@@ -1106 +1106 @@
     'ple.type': 'Type',
     'ple.k_album': 'Album (numbered)',
-    'ple.k_playlist': 'Playlist (track covers)',
+    'ple.k_playlist': 'Playlist (track covers)', 'ple.k_mixtape': 'Mixtape (a tape: forwards and back only)',
     'ple.uitgave': 'Release date',
     'ple.mb_release': 'MusicBrainz release ID',
@@ -1485 +1485 @@
     'apl.none_sub': 'Create one with the button above, or via the 📃 button in the post editor.',
     'apl.pill_playlist': 'playlist',
-    'apl.pill_album': 'album',
+    'apl.pill_album': 'album', 'apl.pill_mixtape': 'mixtape',
     'apl.track': 'track',
     'apl.tracks': 'tracks',
@@ -2115 +2115 @@
     'ple.type': 'Typ',
     'ple.k_album': 'Album (nummeriert)',
-    'ple.k_playlist': 'Playlist (Titelbilder)',
+    'ple.k_playlist': 'Playlist (Titelbilder)', 'ple.k_mixtape': 'Mixtape (Kassette: nur vor und zurück)',
     'ple.uitgave': 'Veröffentlichungsdatum',
     'ple.mb_release': 'MusicBrainz-Release-ID',
@@ -2494 +2494 @@
     'apl.none_sub': 'Erstelle eine über die Schaltfläche oben oder über die 📃-Schaltfläche im Beitrags-Editor.',
     'apl.pill_playlist': 'Playlist',
-    'apl.pill_album': 'Album',
+    'apl.pill_album': 'Album', 'apl.pill_mixtape': 'Mixtape',
     'apl.track': 'Track',
     'apl.tracks': 'Tracks',

src/views/pages/admin-playlists.ejs

@@ -42 +42 @@
       var _aplPlaylist = t('apl.pill_playlist');
       var _aplAlbum    = t('apl.pill_album');
+      var _aplMixtape  = t('apl.pill_mixtape');
+      // Een soort met een eigen teken en een eigen woord. De ternair die hier
+      // stond kende er maar twee, en toonde een mixtape dus als album.
+      var _aplTeken = { album: '💿', playlist: '📃', mixtape: '📼' };
+      var _aplWoord = { album: _aplAlbum, playlist: _aplPlaylist, mixtape: _aplMixtape };
       var _aplTrack    = t('apl.track');
       var _aplTracks   = t('apl.tracks');
@@ -52 +57 @@
         <li class="ax-track" data-id="<%= p.id %>">
           <div class="ax-track-cover" <%- p.cover ? `style="background-image:url('${p.cover}'); background-size:cover; background-position:center"` : '' %>>
-            <% if (!p.cover) { %><span><%= p.kind === 'playlist' ? '📃' : '💿' %></span><% } %>
+            <% if (!p.cover) { %><span><%= _aplTeken[p.kind] || _aplTeken.album %></span><% } %>
           </div>
 
@@ -58 +63 @@
             <div class="ax-track-title">
               <%= p.title %>
-              <span class="ax-pill"><%= p.kind === 'playlist' ? '📃 ' + _aplPlaylist : '💿 ' + _aplAlbum %></span>
+              <span class="ax-pill"><%= (_aplTeken[p.kind] || _aplTeken.album) + ' ' + (_aplWoord[p.kind] || _aplAlbum) %></span>
             </div>
             <div class="ax-track-sub">

src/views/partials/playlist-editor.ejs

@@ -297 +297 @@
 <script type="application/json" data-playlist-editor-i18n><%- JSON.stringify({
   titel: t('ple.titel'), artiest: t('ple.artiest'), jaar: t('ple.jaar'), type: t('ple.type'),
-  k_album: t('ple.k_album'), k_playlist: t('ple.k_playlist'),
+  k_album: t('ple.k_album'), k_playlist: t('ple.k_playlist'), k_mixtape: t('ple.k_mixtape'),
   uitgave: t('ple.uitgave'), mb_release: t('ple.mb_release'),
   cover: t('ple.cover'), cover_kies: t('ple.cover_kies'), cover_url: t('ple.cover_url'),

src/views/shell.ejs

@@ -505 +505 @@
   // Eén nummer voor de hele map. Te vaak bumpen kost één download; te weinig
   // bumpen kost een bugfix die nooit aankomt.
-  var MOD_V = 49;
+  var MOD_V = 50;
 
   // name -> 1 (aan het laden) of de module-namespace (geladen). Een module