Changeset 70dbef4 in Klonkt for src/views/shell.ejs

Timestamp:

08/19/2026 06:15:33 PM (3 weeks ago)

Author:

Bart <bart@…>

Branches:

main

Children:

3a3a882

Parents:

70e9147

git-author:

Bart <bart@…> (08/19/2026 05:44:15 PM)

git-committer:

Bart <bart@…> (08/19/2026 06:15:33 PM)

Message:

De cirkel opent in Grid, en onthoudt dat apart

Grid is daar de standaard: je komt binnen in tegels, niet in de lijst. De knop
blijft een tuimelaar, dus terug naar de lijst kan gewoon.

DIE KEUZE STAAT NU LOS van je gewone voorkeur, en dat is geen extraatje maar
nodig. Op de cirkel betekent 'reader' iets anders dan thuis -- daar bestaat
Lezen niet (het zijn berichten van anderen, van hun servers), dus het valt er
terug op de gewone lijst. Deelden ze één sleutel, dan zou Grid uitzetten op de
cirkel je THUISWEERGAVE veranderen: je komt terug en de feed is niet meer wat je
achterliet. Nu schrijft de cirkel naar pcms-cirkel-view en de rest naar
pcms-feed-view.

En de weergave wordt opnieuw bepaald bij elke htmx-navigatie, niet alleen bij
het laden: van huis naar de cirkel en terug wisselt niet alleen welke knop er
staat maar ook welke onthouden keuze geldt.

Nagelopen in de browser, de hele rondgang: op solo Lezen kiezen, naar de cirkel
(opent in Grid, solo-keuze lekt niet mee), daar Grid uitzetten (lijst,
cirkelsleutel 'list', solo-voorkeur onaangeroerd), en terug thuis staat Lezen er
nog gewoon.

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

File:

• src/views/shell.ejs (modified) (3 diffs)

src/views/shell.ejs

@@ -547 +547 @@
 (function() {
   var body = document.body;
-  // Restore feed view + grid cols from localStorage (overrides server default)
+
+  // DE CIRKEL HEEFT EEN EIGEN KEUZE, met een eigen standaard: Grid.
+  //
+  // Twee redenen om die apart te houden van de gewone voorkeur. Ten eerste is
+  // Lezen daar geen optie (dat zijn berichten van anderen, van hun servers), dus
+  // "reader" betekent er iets anders dan hier: de gewone lijst. Ten tweede zou
+  // een tuimeling op de cirkel anders je SOLO-weergave veranderen -- je zet daar
+  // Grid uit en komt thuis in een andere feed terug dan je achterliet.
+  var CIRKEL_SLEUTEL = 'pcms-cirkel-view';
+  function opCirkel() { return body.classList.contains('on-cirkel'); }
+
+  /** Zet de weergave die bij DEZE pagina hoort. Draait bij init en bij elke nav. */
+  function pasWeergaveToe() {
+    try {
+      if (opCirkel()) {
+        // Alles behalve een uitdrukkelijke 'list' is grid: dat is de standaard.
+        body.dataset.feedView = (localStorage.getItem(CIRKEL_SLEUTEL) === 'list') ? 'reader' : 'grid';
+        return;
+      }
+      var v = localStorage.getItem('pcms-feed-view');
+      // 'timeline' bestaat niet meer als keuze, maar staat nog in de localStorage
+      // van iedereen die hier eerder was. Zonder deze vertaling zou zo iemand een
+      // weergave houden die nergens meer een knop heeft: de pil helemaal grijs, en
+      // een feed die niet is wat hij koos. Lezen is waar Tijdlijn in opging.
+      if (v === 'timeline') v = 'reader';
+      if (v === 'grid' || v === 'reader') body.dataset.feedView = v;
+    } catch (e) { /* geen opslag: de server-standaard blijft staan */ }
+  }
+  pasWeergaveToe();
+
   try {
-    var v = localStorage.getItem('pcms-feed-view');
-    // 'timeline' bestaat niet meer als keuze, maar staat nog in de localStorage
-    // van iedereen die hier eerder was. Zonder deze vertaling zou zo iemand een
-    // weergave houden die nergens meer een knop heeft: de pil helemaal grijs, en
-    // een feed die niet is wat hij koos. Lezen is waar Tijdlijn in opging.
-    if (v === 'timeline') v = 'reader';
-    if (v === 'grid' || v === 'reader') body.dataset.feedView = v;
     var c = parseInt(localStorage.getItem('pcms-grid-cols'), 10);
     if (c === 2 || c === 3 || c === 4) body.dataset.gridCols = String(c);
@@ -587 +609 @@
   // dáár weet syncAria de juiste pagina-class — los van de afterSettle-timing.
   // Een vertraagde herhaling wint eventuele resterende races (OOB-chrome-swap).
-  document.body.addEventListener('htmx:afterSettle', function(){ syncAria(); setTimeout(syncAria, 60); });
-  document.body.addEventListener('pcmsNav', function(){ syncAria(); setTimeout(syncAria, 60); });
+  // Eerst de weergave die bij de NIEUWE pagina hoort, dan pas de pil bijwerken:
+  // van de cirkel naar huis en terug wisselt niet alleen de knop maar ook welke
+  // onthouden keuze telt.
+  document.body.addEventListener('htmx:afterSettle', function(){ pasWeergaveToe(); syncAria(); setTimeout(syncAria, 60); });
+  document.body.addEventListener('pcmsNav', function(){ pasWeergaveToe(); syncAria(); setTimeout(syncAria, 60); });
 
   // Pages where the body actually has a feed to toggle. On these the
@@ -621 +646 @@
       }
       body.dataset.feedView = view;
-      try { localStorage.setItem('pcms-feed-view', view); } catch(_) {}
+      // Op de cirkel onthouden we los, want daar betekent 'reader' de gewone
+      // lijst en hoort het je thuisweergave niet aan te raken.
+      try {
+        if (opCirkel()) localStorage.setItem(CIRKEL_SLEUTEL, view === 'grid' ? 'grid' : 'list');
+        else localStorage.setItem('pcms-feed-view', view);
+      } catch(_) {}
       syncAria();
       // On non-feed pages the switcher acts as a navigation: take the