Changeset 42e7616 in Klonkt

Timestamp:

07/26/2026 10:00:36 AM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

de89079

Parents:

ef1853c

Message:

Fix: load-more toont ineens de Klonkt-header (chrome-swap bij append)

Op Messages (en ook News/Cirkel) verscheen na "Load more" + omhoog scrollen de
volledige Klonkt-header. Oorzaak: renderPage behandelde de load-more-append als
een HTMX-NAVIGATIE en stuurde de HX-Trigger pcmsNav (met bodyClass) + de
out-of-band chrome mee. De append-render geeft geen bodyClass mee, dus die viel
terug op de default 'on-home'; de pcmsNav-listener in shell.ejs stript dan
'on-special' en de OOB-chrome herbouwt de header voor de home-variant → de
verborgen header komt tevoorschijn.

Een load-more voegt alleen rijen toe aan de bestaande pagina; dat is geen
navigatie. renderPage stuurt voor een append (req.query.append === '1') nu enkel
de content, zonder de nav-HX-Trigger en zonder de OOB-chrome. De OOB load-more-
knop zit in de content zelf en blijft dus gewoon werken.

Changed files:
src/middleware/render.js

• isPartial + append=1 -> alleen content sturen (geen pcmsNav-trigger/OOB-chrome)

remarks: 187 tests groen. Dekt messages, news en cirkel load-more. Bug zit achter
login, dus niet live in een preview te reproduceren; diagnose via de code
bevestigd (pcmsNav-listener shell.ejs + on-special + ontbrekende bodyClass).

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

File:

• src/middleware/render.js (modified) (1 diff)

src/middleware/render.js

@@ -210 +210 @@
 
     if (isPartial) {
+      // A "Load more" append (hx-swap=beforeend into a sub-list) is NOT a
+      // navigation: it only adds rows to the existing page. It must NOT touch
+      // the site chrome or the body class. Emitting the nav HX-Trigger + OOB
+      // chrome here (below) rebuilds the header for the DEFAULT bodyClass —
+      // which, on a 'on-special' page like Messages, swaps in the full Klonkt
+      // header that the page had hidden. So for an append, send content only.
+      if (req.query.append === '1') {
+        return res.send(injectCspNonce(pageContent, res.locals.cspNonce));
+      }
       // HTMX: just send the content. Set HX-Trigger for body class swap.
       // HTTP-header values are Latin-1 only — a title with an em-dash, smart