Changeset daa8eb8 in Klonkt

Timestamp:

08/16/2026 01:09:45 PM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

5280532

Parents:

a982e7e

Message:

Beheer: de terugknop op alle vijf dezelfde maat

Robin zag het (16-8): de knop naar /admin verschilde van grootte. Drie
pagina's kregen bij de omzetting class="ax-btn", audio en playlists
hielden de globale class="btn". Dat scheelt zichtbaar --

.btn padding 0.55rem 1rem · radius 5 · geen minimumhoogte
.ax-btn padding 0.6rem 1.1rem · radius 8 · min-height 44px

Het wordt .ax-btn: dit zijn ax-pagina's, en 44px is de maat waar een
vinger op past.

Nu een partial, voor een regel van niets, om dezelfde reden als bij de
tabbalk: een gedeeld element dat op elke pagina los staat loopt uiteen.
Dit is de tweede keer vandaag dat dat gebeurt, en de eerste keer had ik
het kunnen zien -- ik heb de tabbalk toen wel gedeeld gemaakt en de knop
ernaast niet.

scripts/render-admin-views.mjs dekt nu alle vijf de pagina's en
controleert de terugknop op zijn KLASSENAAM. Dat verschil ziet er in de
bron uit als hetzelfde, dus een controle die alleen kijkt of er een link
staat zou het niet vinden. Nagelopen dat hij kan falen: met .btn erin
meldt hij 10 stuk, hersteld nul.

Opmaak nagemeten met admin-css-gelijk.mjs: geen regel verdwenen, geen
regel veranderd, geen regel erbij op alle vijf.

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

Files:

• scripts/render-admin-views.mjs (modified) (3 diffs)
• src/views/pages/admin-audio.ejs (modified) (1 diff)
• src/views/pages/admin-listeners.ejs (modified) (1 diff)
• src/views/pages/admin-media.ejs (modified) (1 diff)
• src/views/pages/admin-playlists.ejs (modified) (1 diff)
• src/views/pages/admin-videos.ejs (modified) (1 diff)
• src/views/partials/admin-back.ejs (added)

scripts/render-admin-views.mjs

@@ -27 +27 @@
 
 const gevuld = {
+  'admin-audio': { tracks: [], playlists: [], maxBytesMb: 20, maxWavMb: 100 },
+  'admin-playlists': { playlists: [] },
   'admin-media': { items: [{ file: 'a.jpg', url: '/m/a.jpg', kb: 12, usedCount: 0, hasVideo: true }], unusedCount: 1 },
   'admin-videos': { items: [{ file: 'b.mp4', url: '/m/b.mp4', kb: 99, usedCount: 2 }] },
@@ -32 +34 @@
 };
 const leeg = {
+  'admin-audio': { tracks: [], playlists: [], maxBytesMb: 20, maxWavMb: 100 },
+  'admin-playlists': { playlists: [] },
   'admin-media': { items: [], unusedCount: 0 },
   'admin-videos': { items: [] },
@@ -54 +58 @@
       if (!html.includes('.ax-tab[aria-current')) fouten.push('admin-styles ontbreekt');
       if (!html.includes('class="container ax-page"')) fouten.push('ax-page-romp ontbreekt');
+      // De terugknop, met NAAM en al. Robin zag op 16-8 dat hij van grootte
+      // verschilde: drie pagina's hadden ax-btn en twee de globale btn, en dat
+      // scheelt 10px hoogte en een andere radius. Dit is de regel die dat
+      // vasthoudt -- de klasse staat er letterlijk in, want juist zo'n verschil
+      // ziet er in de bron uit als hetzelfde.
+      const terug = /<a href="\/admin" class="([^"]+)">/.exec(markup);
+      if (!terug) fouten.push('geen terugknop naar /admin');
+      else if (terug[1] !== 'ax-btn') fouten.push('terugknop heeft class="' + terug[1] + '" i.p.v. ax-btn');
       // Twee vormen van stille schade die er goed uitzien in de bron.
       if (markup.includes("[object Promise]")) fouten.push('onopgeloste include');

src/views/pages/admin-audio.ejs

@@ -5 +5 @@
 <div class="container ax-page">
 
-  <p><a href="/admin" class="btn">&larr; <%= t('nav.admin') %></a></p>
+  <%- include('../partials/admin-back') %>
   <%- include('../partials/media-tabs', { active: 'audio', audioOn: true }) %>
   <header class="ax-header">

src/views/pages/admin-listeners.ejs

@@ -8 +8 @@
 <div class="container ax-page">
 
-  <p><a href="/admin" class="ax-btn">&larr; <%= t('nav.admin') %></a></p>
+  <%- include('../partials/admin-back') %>
   <%- include('../partials/media-tabs', { active: 'listeners', audioOn: audioOn }) %>
   <header class="ax-header">

src/views/pages/admin-media.ejs

@@ -8 +8 @@
 <div class="container ax-page">
 
-  <p><a href="/admin" class="ax-btn">&larr; <%= t('nav.admin') %></a></p>
+  <%- include('../partials/admin-back') %>
   <%- include('../partials/media-tabs', { active: 'images', audioOn: audioOn }) %>
   <header class="ax-header">

src/views/pages/admin-playlists.ejs

@@ -11 +11 @@
 <div class="container ax-page">
 
-  <p><a href="/admin" class="btn">&larr; <%= t('nav.admin') %></a></p>
+  <%- include('../partials/admin-back') %>
   <%- include('../partials/media-tabs', { active: 'playlists', audioOn: true }) %>
   <header class="ax-header">

src/views/pages/admin-videos.ejs

@@ -10 +10 @@
 <div class="container ax-page">
 
-  <p><a href="/admin" class="ax-btn">&larr; <%= t('nav.admin') %></a></p>
+  <%- include('../partials/admin-back') %>
   <%- include('../partials/media-tabs', { active: 'videos', audioOn: audioOn }) %>
   <header class="ax-header">