Changeset 520c993 in Klonkt

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

Files:

• src/assets/css/style.css (modified) (1 diff)
• src/config/database.js (modified) (1 diff)
• src/routes/admin-sites.js (modified) (2 diffs)
• src/services/i18n.js (modified) (3 diffs)
• src/views/pages/admin-site-edit.ejs (modified) (1 diff)
• src/views/shell.ejs (modified) (3 diffs)
• test/feed-alt-view.test.js (modified) (1 diff)

src/assets/css/style.css

@@ -2549 +2549 @@
 .feed-reader .read-post:last-child .read-anker { scroll-snap-align: none; }
 
+/* ELK BERICHT EEN EIGEN SCHERM, als de site dat wil (Instellingen -> Weergave).
+   Aan: een kort bericht vult het scherm en je ziet het volgende pas na een snap.
+   Uit (standaard): de stroom is bericht na bericht, zonder lege ruimte eronder.
+   Dit stond eerst vast op AAN, is toen vast op UIT gezet omdat een kort bericht
+   een halve pagina leegte gaf onder "Replies and reactions", en is nu een keuze
+   -- want welke van de twee prettiger leest hangt af van wat je schrijft. */
+body[data-reader-pages="1"] .feed-reader .read-post { min-height: 100svh; }
+
 /* Een lijn tussen twee berichten: zonder grens is een stroom een lange brij. */
 .feed-reader .read-post + .read-post { border-top: 1px solid color-mix(in srgb, currentColor 12%, transparent); }

src/config/database.js

@@ -209 +209 @@
   // het is een mediaquery, dus geen resize-afhandeling en geen flikkering.
   ensureColumn('sites', 'feed_alt_view', "TEXT DEFAULT 'reader'");   // reader | timeline | auto
+  // Krijgt elk bericht in Lezen zijn EIGEN scherm? Aan betekent min-height:100svh
+  // per bericht: een kort bericht vult dan het scherm en je ziet het volgende
+  // pas na een snap. Uit is de stroom zoals hij nu is, bericht na bericht zonder
+  // lege ruimte. Stond eerst vast op AAN, is toen vast op UIT gezet omdat een
+  // kort bericht een halve pagina leegte gaf; nu is het een keuze.
+  ensureColumn('sites', 'reader_full_page', 'INTEGER DEFAULT 0');
   ensureColumn('sites', 'show_search',     'INTEGER DEFAULT 1');
   ensureColumn('sites', 'show_archive_link', 'INTEGER DEFAULT 1');

src/routes/admin-sites.js

@@ -371 +371 @@
       enable_audio_player = ?,
       approve_followers = ?,
-      feed_view_default = ?, feed_view_switch = ?, feed_alt_view = ?,
+      feed_view_default = ?, feed_view_switch = ?, feed_alt_view = ?, reader_full_page = ?,
       show_search = ?, show_archive_link = ?,
       custom_css = ?, custom_head_html = ?, custom_foot_html = ?,
@@ -396 +396 @@
     f.feed_view_switch ? 1 : 0,
     feedAlt,
+    f.reader_full_page ? 1 : 0,
     f.show_search ? 1 : 0,
     f.show_archive_link ? 1 : 0,

src/services/i18n.js

@@ -33 +33 @@
     'switch.solo': 'Solo',
     'switch.circle': 'Cirkels',
-    'switch.grid': 'Grid', 'switch.reader': 'Lezen', 'switch.timeline': 'Tijdlijn', 'read.to_top': 'Terug naar boven', 'read.pinned': 'Vastgepind',
+    'switch.grid': 'Grid', 'switch.reader': 'Lezen', 'switch.timeline': 'Tijdlijn', 'read.to_top': 'Terug naar boven', 'read.pinned': 'Vastgepind', 'asite.reader_full_page': 'Lezen: elk bericht een eigen scherm',
     'asite.feed_alt': 'Tweede weergave', 'asite.feed_alt_reader': 'Lezen', 'asite.feed_alt_timeline': 'Tijdlijn', 'asite.feed_alt_auto': 'Lezen op mobiel, Tijdlijn op desktop',
     'switch.reader_solo_only': 'Lezen kan alleen in Solo — in de cirkel staan berichten van anderen',
@@ -1049 +1049 @@
     'switch.solo': 'Solo',
     'switch.circle': 'Circles',
-    'switch.grid': 'Grid', 'switch.reader': 'Reader', 'switch.timeline': 'Timeline', 'read.to_top': 'Back to top', 'read.pinned': 'Pinned',
+    'switch.grid': 'Grid', 'switch.reader': 'Reader', 'switch.timeline': 'Timeline', 'read.to_top': 'Back to top', 'read.pinned': 'Pinned', 'asite.reader_full_page': 'Reader: one screen per post',
     'asite.feed_alt': 'Second view', 'asite.feed_alt_reader': 'Reader', 'asite.feed_alt_timeline': 'Timeline', 'asite.feed_alt_auto': 'Reader on mobile, Timeline on desktop',
     'switch.reader_solo_only': 'Reader is Solo only — a circle shows other people\u2019s posts',
@@ -2056 +2056 @@
     'switch.solo': 'Solo',
     'switch.circle': 'Zirkel',
-    'switch.grid': 'Raster', 'switch.reader': 'Lesen', 'switch.timeline': 'Zeitleiste', 'read.to_top': 'Nach oben', 'read.pinned': 'Angeheftet',
+    'switch.grid': 'Raster', 'switch.reader': 'Lesen', 'switch.timeline': 'Zeitleiste', 'read.to_top': 'Nach oben', 'read.pinned': 'Angeheftet', 'asite.reader_full_page': 'Lesen: ein Bildschirm pro Beitrag',
     'asite.feed_alt': 'Zweite Ansicht', 'asite.feed_alt_reader': 'Lesen', 'asite.feed_alt_timeline': 'Zeitleiste', 'asite.feed_alt_auto': 'Lesen mobil, Zeitleiste am Desktop',
     'switch.reader_solo_only': 'Lesen gibt es nur in Solo — im Zirkel stehen Beitr\u00e4ge anderer',

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' : '' %>>

src/views/shell.ejs

@@ -222 +222 @@
 
 <!-- v9 stylesheet (full palette system) -->
-<link rel="stylesheet" href="/assets/css/style.css?v=82">
+<link rel="stylesheet" href="/assets/css/style.css?v=83">
 <script>
 /* iOS safe-area, built by hand. env(safe-area-inset-top) resolves to 0 on this iOS in
@@ -345 +345 @@
 </head>
 
-<body class="<%= bodyClass || 'on-home' %> has-bottom-tab" data-js="chrome<%= (typeof pageJs !== 'undefined' && pageJs) ? ' ' + pageJs : '' %>" data-feed-view="<%- _e(safeSite.feed_view_default === 'grid' ? 'grid' : (["timeline","auto"].indexOf(safeSite.feed_alt_view) >= 0 ? safeSite.feed_alt_view : "reader")) %>" data-feed-alt="<%- _e(["timeline","auto"].indexOf(safeSite.feed_alt_view) >= 0 ? safeSite.feed_alt_view : "reader") %>" data-grid-cols="3" data-site-base="<%- _e((typeof siteUrlBase !== 'undefined' && siteUrlBase) ? siteUrlBase : '') %>">
+<body class="<%= bodyClass || 'on-home' %> has-bottom-tab" data-js="chrome<%= (typeof pageJs !== 'undefined' && pageJs) ? ' ' + pageJs : '' %>" data-feed-view="<%- _e(safeSite.feed_view_default === 'grid' ? 'grid' : (["timeline","auto"].indexOf(safeSite.feed_alt_view) >= 0 ? safeSite.feed_alt_view : "reader")) %>" data-feed-alt="<%- _e(["timeline","auto"].indexOf(safeSite.feed_alt_view) >= 0 ? safeSite.feed_alt_view : "reader") %>" data-reader-pages="<%- safeSite.reader_full_page ? 1 : 0 %>" data-grid-cols="3" data-site-base="<%- _e((typeof siteUrlBase !== 'undefined' && siteUrlBase) ? siteUrlBase : '') %>">
 
 <% if (typeof isViewer !== 'undefined' && isViewer) { %>
@@ -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 = 20;
+  var MOD_V = 21;
 
   // name -> 1 (aan het laden) of de module-namespace (geladen). Een module

test/feed-alt-view.test.js

@@ -102 +102 @@
 // op; valt er een test om, dan hangt de hele suite. Zo overkwam het me bij de
 // controleproef -- de test die de fout moest aantonen, hing erop.
+// ── Elk bericht een eigen scherm in Lezen (Robin, 20-8) ─────────────────────
+// Zelfde reden als hierboven om door de ECHTE route te gaan: dit voegt een
+// kolom toe aan dezelfde UPDATE, en juist daar ging het eerder mis.
+test('de nieuwe kolom staat standaard uit', () => {
+  const s = siteVan('proef');
+  assert.equal(s.reader_full_page, 0, 'bestaande sites veranderen niet van gedrag');
+});
+
+test('aanzetten werkt, en de buren schuiven niet mee', async () => {
+  await post('/admin/sites/proef/save', {
+    title: 'Proef', feed_alt_view: 'timeline', feed_view_default: 'grid',
+    feed_view_switch: '1', reader_full_page: '1', show_search: '1', show_archive_link: '1',
+  });
+  const s = siteVan('proef');
+  assert.equal(s.reader_full_page, 1);
+  // De kolommen rond de nieuwe in de UPDATE. Zet je de kolom er wel bij en de
+  // waarde niet, dan schuift alles erna een plek op en is dit het eerste dat
+  // omvalt.
+  assert.equal(s.feed_alt_view, 'timeline', 'de kolom ervoor');
+  assert.equal(s.show_search, 1, 'de kolom erna');
+  assert.equal(s.show_archive_link, 1, 'en die daarachter');
+  assert.equal(s.feed_view_switch, 1);
+});
+
+test('en weer uit', async () => {
+  await post('/admin/sites/proef/save', { title: 'Proef', feed_alt_view: 'reader', feed_view_default: 'reader' });
+  assert.equal(siteVan('proef').reader_full_page, 0, 'een niet-aangevinkt vakje zet hem uit');
+});
+
 test.after(() => { server.closeAllConnections?.(); server.close(); });