Changeset 749605a in Klonkt for src/views/partials/playlist-editor.ejs

Timestamp:

08/16/2026 03:06:18 PM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

a5caa73

Parents:

a5515b4 (diff), 9026de5 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Bibliotheek en album-federatie terug op main (shaer-0nh, shaer-756s)

Achttien commits, in drie brokken.

DE BIBLIOTHEEK. Een Library-skelet met een eigen volgerssoort: wie de
bibliotheek volgt krijgt de muziek en met opzet niet de gewone posts, en
die staan daarom in een eigen tabel -- een vlag die iemand vergeet te
filteren bezorgt stilletjes de Krant aan mensen die er niet om vroegen.
Daarna wat er nodig bleek om werkelijk gelezen te worden: audience
(zonder dat gaf hun create() een KeyError en dus een 500 op onze URL),
de ongeordende collectievorm, en paginering op elke collectie die first
belooft -- die laatste vond meteen dat featured hetzelfde gat had.

HET VOCABULAIRE. fw:track in de kleine vorm die Emissary ook stuurt,
artist_credit met de site-actor als Artist (mogelijk geworden door de
MusicBrainz-koppeling), en het album als ingesloten fw:Album met
released, musicbrainzId en een eigen artist_credit. Daarvoor kwamen er
twee kolommen op playlists, alleen bij kind='album' en afgedwongen in
PlaylistService -- niet in het scherm, want de API ligt open.

HET BEHEER. De vijf mediapagina's stonden op drie verschillende
ontwerpen; nu op een gedeelde ax-romp, met de primitieven op een plek in
plaats van in acht kopieen. De playlist-modal ging alsnog door i18n: die
teksten stonden hard in het script, wat niet opviel omdat het toevallig
de brontaal was.

WAT ER NIET IN ZIT: bitrate en size top-level als fw:. Dat is bewust
(shaer-3f8a). Hun eigen outbox zet ze op de Link, net als wij, en het
wordt een gesprek in plaats van een aanpassing -- briefing in
~/bewaard/funkwhale-briefing-en.md.

Gereedschap dat meekomt: scripts/funkwhale-gat.mjs meet het gat naar hun
ingest, admin-css.mjs en admin-css-gelijk.mjs ontleden en vergelijken de
beheer-CSS, render-admin-views.mjs rendert de views gevuld en leeg. Die
laatste drie omdat de testsuite geen enkele view aanraakt.

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

File:

• src/views/partials/playlist-editor.ejs (modified) (2 diffs)

src/views/partials/playlist-editor.ejs

@@ -99 +99 @@
 }
 .pl-field { display: flex; flex-direction: column; gap: .25rem; }
+/* `display: flex` wint van het hidden-attribuut, dus zonder deze regel doet
+   [hidden] op een .pl-field NIETS -- het veld blijft gewoon staan. De
+   uitgavevelden schakelen daarop (album versus afspeellijst). */
+.pl-field[hidden] { display: none; }
 .pl-field-full { grid-column: 1 / -1; }
 .pl-field > span {
@@ -280 +284 @@
 </style>
 
-<%# Het script staat in assets/js/mod/playlist-editor.js; de gegevens via partials/page-data.ejs (shaer-bqr). %>
+<%# Het script staat in assets/js/mod/playlist-editor.js.
+
+    EIGEN GEGEVENSBLOK, niet dat van partials/page-data.ejs. pageData() pakt met
+    querySelector EEN blok, en deze modal wordt ingesloten op pagina's die er al
+    zelf een hebben (admin-playlists, post-edit) -- een tweede blok zou daar
+    stilzwijgend genegeerd worden en dan stond de hele modal weer in het
+    Nederlands. Eigen kenmerk, eigen lezer.
+
+    De teksten stonden hier tot 16-8 hard in het script. Dat viel niet op omdat
+    het toevallig de brontaal was: een Engelse of Duitse beheerder kreeg een
+    Nederlandse modal midden in een vertaalde pagina. %>
+<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'),
+  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'),
+  tracks_in: t('ple.tracks_in'), sleep_hint: t('ple.sleep_hint'),
+  beschikbaar: t('ple.beschikbaar'), zoek: t('ple.zoek'), geen_res: t('ple.geen_res'),
+  leeg_sel: t('ple.leeg_sel'),
+  t_edit: t('ple.t_edit'), t_new: t('ple.t_new'), dialoog: t('ple.dialoog'),
+  sluiten: t('ple.sluiten'), annuleren: t('ple.annuleren'),
+  opslaan: t('ple.opslaan'), aanmaken: t('ple.aanmaken'),
+  bezig_opslaan: t('ple.bezig_opslaan'),
+  versleep: t('ple.versleep'), verwijder: t('ple.verwijder'),
+  geen_audio: t('ple.geen_audio'), e_geen_tracks: t('ple.e_geen_tracks'),
+  e_tracks: t('ple.e_tracks'), e_opslaan: t('ple.e_opslaan'), e_mislukt: t('ple.e_mislukt'),
+  e_alleen_afb: t('ple.e_alleen_afb'), bezig: t('ple.bezig'), e_upload: t('ple.e_upload'),
+}).replace(/</g, '\\u003c') %></script>