Changeset 77019fa in Klonkt for src/assets/js/mod/read.js

Timestamp:

08/20/2026 10:24:40 AM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

96a99bf

Parents:

22d2dba5

Message:

Reader: the last post snaps too

Its top used to be unreachable — too little page below it to bring it to
the top edge (measured: snap point 2548, max scroll 2356) — so the
browser fell back to the nearest reachable position, the bottom. The
previous answer was to drop its snap point; this is the real one: the
LAST post gets a full screen height, which makes that top exactly
reachable (shorter posts grow to fit, taller ones already were). Costs
nothing visually — posts have no frame, only a separator above.

lvh, not svh: with a disappearing address bar the visible screen is
sometimes the large viewport, and svh would fall just short.

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

File:

• src/assets/js/mod/read.js (modified) (1 diff)

src/assets/js/mod/read.js

@@ -392 +392 @@
     easing: (t) => 1 - Math.pow(1 - t, 4),
   });
-  // Het snappunt is de bovenkant van elk bericht. Het laatste doet niet mee:
-  // zijn bovenkant is niet te bereiken, er zit te weinig pagina onder.
+  // Het snappunt is de bovenkant van elk bericht -- ook van het laatste. Dat
+  // deed eerder niet mee omdat zijn bovenkant onbereikbaar was; sinds het
+  // laatste bericht in CSS een volle schermhoogte krijgt, kan hij wel.
   const berichten = [...document.querySelectorAll('.feed-reader .read-post')];
-  berichten.slice(0, -1).forEach((a) => snap.addElement(a, { align: 'start' }));
+  berichten.forEach((a) => snap.addElement(a, { align: 'start' }));
 
   // ALLEEN VOORUIT VANGEN. Dit hoort bij de ruime zone hierboven en is niet