Changeset 7cf8144 in Klonkt for src/routes/posts.js

Timestamp:

08/18/2026 06:47:58 PM (3 weeks ago)

Author:

Bart <bart@…>

Branches:

main

Children:

9da8027

Parents:

2d2fd8c

git-author:

Bart <bart@…> (08/18/2026 05:28:31 AM)

git-committer:

Bart <bart@…> (08/18/2026 06:47:58 PM)

Message:

Lezen is een weergave van de feed, geen eigen plek

/read was een route met een eigen shell: eigen pagina, eigen chrome-gedrag, en
op een site zonder berichten een 404 -- een dode knop in de switcher. Nu is het
wat het altijd had moeten zijn: een derde waarde van body[data-feed-view], naast
timeline en grid. Dezelfde berichten, andere vorm.

De feed stuurt ze alledrie mee en CSS kiest, precies zoals timeline/grid dat al
deden. Het lijf loopt door PostAccessService, dus een gesloten poort levert ook
hier geen tekst op. "Meer laden" vult de leesstroom OOB mee aan, net als de
tegels.

DE LEGE STAAT zit in de :has(.feed-reader) op de verbergregel. Zonder berichten
rendert de leessectie niet, en dan blijft de tijdlijn staan -- want daar staat
"nog niets geschreven" met de knop om te beginnen. Zonder die voorwaarde was
Lezen op een verse site een wit scherm. Feeds zonder leessectie (cirkel,
archief) houden om dezelfde reden gewoon hun tijdlijn.

Wat weg kon, en dat is de winst:

• de route, pages/read.ejs, en 'on-read' als page-class;
• mod/read.js van 222 naar 51 regels. Het ophalen van buren, het corrigeren van de scrollpositie bij invoegen en het onderbreken van het snappen zijn niet opgelost maar OVERBODIG: de feed levert de berichten al. Wat blijft is de tik.
• read.end en read.load_error: het script plaatst geen zinnen meer.

De balken schuiven niet meer weg. Dat was mooi op een eigen scherm, maar hier
zou het de switcher verbergen waarmee je net koos -- en dan kun je niet terug.

MOD_V naar 4, style.css naar v70.

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

File:

• src/routes/posts.js (modified) (4 diffs)

src/routes/posts.js

@@ -232 +232 @@
 
   if (append) {
-    return renderPage(req, res, 'partials/home-append', { posts, hasMore, nextOffset: offset + FEED_PAGE, moreBase });
+    return renderPage(req, res, 'partials/home-append', {
+      posts, hasMore, nextOffset: offset + FEED_PAGE, moreBase,
+      readerItems: readerItems(site, posts, req),
+    });
   }
 
@@ -243 +246 @@
     pinnedPosts,
     posts,
+    readerItems: readerItems(site, [...pinnedPosts, ...posts], req),
     hasMore, nextOffset: offset + FEED_PAGE, moreBase,
     movedTo,
@@ -249 +253 @@
     socialDescr: site.description || site.tagline || '',
     bodyClass: 'on-home',
+    // mod/read.js: alleen nog de tik-op-een-bericht in de leesweergave.
+    pageJs: 'read',
   });
-});
-
-// ==================== LEES (meeslepende tijdlijn) ====================
-//
-// Eén bericht vult het scherm, de chrome schuift weg, en je scrollt voorbij de
-// rand naar het vorige of volgende. Naar bartoverkamp.nl, waar prev/next
-// schildwachten boven en onder het artikel staan en het overscrollen zelf de
-// navigatie is.
-//
-// De buren komen uit postNeighbors(), dezelfde die de gewone postpagina al
-// gebruikt -- dus "vorige" en "volgende" betekenen hier precies hetzelfde als
-// daar, ook voor gepinde berichten.
-//
-// ?partial=1 levert alleen het artikel, want dat is wat de schildwacht inruilt.
-router.get('/read/:slug?', (req, res, next) => {
-  const site = res.locals.site;
-  if (!site) return next();
-
-  // Zonder slug: het nieuwste bericht, zodat /read een ingang is en niet een
-  // fout. Gepind eerst, net als op de voorpagina.
-  const post = req.params.slug
-    ? db.prepare(`SELECT p.*, u.username as author_username FROM posts p JOIN users u ON p.author_id = u.id
-                  WHERE p.site_id = ? AND p.slug = ?`).get(site.id, req.params.slug)
-    : db.prepare(`SELECT p.*, u.username as author_username FROM posts p JOIN users u ON p.author_id = u.id
-                  WHERE p.site_id = ? AND p.status = 'published'
-                  ORDER BY (p.pinned = 0) ASC, p.pinned ASC, p.published_at DESC LIMIT 1`).get(site.id);
-  if (!post) return next();
-
-  // Het besluit en het lijf komen uit één plek (PostAccessService). Een dichte
-  // poort levert hier GEEN tekst op: die wordt niet eens gerenderd.
-  const _u = req.query.u ? verifyBlob(String(req.query.u)) : null;
-  const unlockedSlug = (_u && _u.purpose === 'unlocked' && _u.siteId === site.id) ? String(_u.post) : null;
-  const entry = postEntry(post, { user: req.session?.user || null, site, unlockedSlug },
-    { renderBody: (p) => renderPostBodyHtml(site, p, req) });
-
-  if (entry.access === 'forbidden') return next();
-
-  const { newerPost, olderPost } = postNeighbors(site, post);
-  const model = {
-    post, entry, newerPost, olderPost,
-    pageTitle: post.title || site.title,
-    // on-read zet de chrome weg; de mini-topnav blijft (chrome.ejs, _headerless).
-    bodyClass: 'on-read on-special',
-    pageJs: 'read',
-  };
-  // ALLEEN de leesmodule krijgt één artikel; ?fragment=1 is haar eigen teken.
-  //
-  // Hier stond `?partial=1 || hx-request`, en dat waren twee dingen door elkaar.
-  // Op deze site betekent ?partial=1 al iets anders -- "de pagina zonder de
-  // schil", voor htmx -- dus een gebooste link naar /read kreeg één kaal
-  // artikel in #pcms-main geschoven: geen stroom, geen on-read, geen script.
-  // Onzichtbaar zolang je /read alleen intikt; zichtbaar zodra er een link naar
-  // toe bestaat (de Lezen-knop in de view-switcher).
-  //
-  // Alles wat GEEN fragment vraagt gaat naar pages/read, en renderPage bepaalt
-  // daar zelf of de schil eromheen moet -- net als bij elke andere pagina.
-  if (req.query.fragment === '1') {
-    return renderPage(req, res, 'partials/read-article', model);
-  }
-  recordPageview(site.id, req);
-  return renderPage(req, res, 'pages/read', model);
 });
 
@@ -748 +693 @@
 // Renders a post's display HTML: baked content + the dynamic audio/embed layer.
 // Extracted so the paid unlock (slice 4) serves the exact same body as the page.
+// Dezelfde berichten, klaar voor de leesweergave.
+//
+// Tijdlijn en Grid tonen kaartjes; Lezen toont het hele stuk. Het is dus geen
+// andere PAGINA maar een andere vorm van dezelfde rijen -- vandaar dat de feed
+// ze alledrie meestuurt en CSS kiest, precies zoals timeline/grid dat al deden.
+//
+// Het lijf loopt door PostAccessService: een gesloten poort levert hier GEEN
+// tekst op, want wat niet gerenderd wordt kan ook niet lekken.
+function readerItems(site, rows, req) {
+  const viewer = { user: req.session?.user || null, site, unlockedSlug: null };
+  return rows.map((post) => ({
+    post,
+    entry: postEntry(post, viewer, { renderBody: (p) => renderPostBodyHtml(site, p, req) }),
+  }));
+}
+
 export function renderPostBodyHtml(site, post, req) {
   let html = (post.content_rendered != null && post.content_rendered !== '')