Changeset 520c993 in Klonkt for test

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:

• test/feed-alt-view.test.js (modified) (1 diff)

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(); });