Changeset 520c993 in Klonkt for src/views/pages/admin-site-edit.ejs

Timestamp:

08/20/2026 02:46:09 AM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

bc9ef3b, c581e5e

Parents:

88838bc

git-author:

Robin <roboburr@…> (08/20/2026 02:42:45 AM)

git-committer:

Robin <roboburr@…> (08/20/2026 02:46:09 AM)

Message:

Lezen: elk bericht een eigen scherm wordt een keuze

Robins vraag (20-8): de lege ruimtes tussen berichten optioneel maken, en
specifiek voor Lezen.

Nieuwe kolom reader_full_page op sites, standaard UIT zodat bestaande
sites niet ineens anders lezen:

uit bericht na bericht, geen lege ruimte eronder
aan elk bericht vult minstens een scherm (min-height:100svh)

Dit stond eerst vast op AAN, is op 20-8 vast op UIT gezet omdat een kort
bericht een halve pagina leegte gaf onder "Replies and reactions", en hoort
dus eigenlijk bij de schrijver: lange stukken lezen prettiger in de stroom,
korte losse berichten juist als eigen scherm.

De keuze staat in Beheer -> Instellingen bij de andere feed-keuzes, en gaat
via een attribuut op body zodat CSS het werk doet.

OVER DE SQL, want daar ging het deze week twee keer mis: deze wijziging
voegt een kolom toe aan dezelfde UPDATE waar ik eerder een kolom zonder
waarde in zette, waarna show_search de waarde van zijn buurman kreeg en
alles erna een plek opschoof. De tests gaan daarom door de ECHTE route en
toetsen expliciet de buurkolommen. Controleproef gedaan: laat je de waarde
weg, dan vallen 6 van de 8 tests om.

style.css naar v83, MOD_V naar 21.

Suite 1172/1172.

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

File:

• src/views/pages/admin-site-edit.ejs (modified) (1 diff)

src/views/pages/admin-site-edit.ejs

@@ -218 +218 @@
         <span><%= t('asite.feed_switch') %></span>
       </label>
+      <%# Alleen van toepassing op Lezen: geeft elk bericht daar een eigen scherm. %>
+      <label class="cb">
+        <input type="checkbox" name="reader_full_page" value="1" <%= site.reader_full_page ? 'checked' : '' %>>
+        <span><%= t('asite.reader_full_page') %></span>
+      </label>
       <label class="cb">
         <input type="checkbox" name="show_search" value="1" <%= site.show_search ? 'checked' : '' %>>