Changeset 3a3a882 in Klonkt for src/views/shell.ejs

Timestamp:

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

Author:

Robin <roboburr@…>

Branches:

main

Children:

8c91ed3

Parents:

70dbef4

Message:

Lezen staat in de cirkel zichtbaar maar dood, in plaats van weg

De knop verdween daar helemaal, met het argument dat een knop die niets
doet erger is dan geen knop. In de praktijk pakte dat andersom uit: de
pil werd smaller op de cirkel en breder op solo, en die sprong leest als
'er is hier iets weg' -- terwijl juist NIET duidelijk werd dat Lezen
bestaat en er alleen hier niet kan. Nu staat hij er grijs bij, met een
titel die zegt waarom, en is Grid zichtbaar de enige mogelijkheid.

Wat daarmee vervalt: de aparte cirkel-voorkeur ('pcms-cirkel-view').
Die kon nog maar een waarde hebben. De solo-keuze blijft onaangeroerd --
op de cirkel schrijven we niets meer naar localStorage, dus je komt thuis
in de weergave die je achterliet.

Drie plekken rekenden erop dat de knop er niet was en zijn meegegaan: de
terugval 'is dit de enige knop, dan brengt hij je naar de standaard' (er
zijn er nu twee, en op de cirkel hoort Grid juist de enige te blijven),
de aria-sync (een dode knop kan nooit de actieve zijn) en de klik-
afhandeling (expliciet op sw.disabled, niet vertrouwen op dat een
disabled button geen click vuurt -- deze listener hangt op document).

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

File:

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

src/views/shell.ejs

@@ -548 +548 @@
   var body = document.body;
 
-  // DE CIRKEL HEEFT EEN EIGEN KEUZE, met een eigen standaard: Grid.
+  // DE CIRKEL HEEFT MAAR EEN WEERGAVE: 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';
+  // Lezen is daar geen optie -- dat zijn berichten van ANDEREN, van hun servers,
+  // en "het hele stuk" hebben wij daar niet in handen. De knop staat er wel,
+  // maar dood (zie view-switcher.ejs). Er valt dus niets te kiezen en dus ook
+  // niets te onthouden: de eigen cirkel-voorkeur die hier stond
+  // ('pcms-cirkel-view') is vervallen, want hij kon nog maar een waarde hebben.
+  //
+  // Wat WEL blijft: de cirkel raakt je solo-keuze niet aan. Die staat in
+  // 'pcms-feed-view' en wordt hier nergens geschreven, zodat je thuiskomt in de
+  // weergave die je achterliet.
   function opCirkel() { return body.classList.contains('on-cirkel'); }
 
@@ -562 +565 @@
     try {
       if (opCirkel()) {
-        // Alles behalve een uitdrukkelijke 'list' is grid: dat is de standaard.
-        body.dataset.feedView = (localStorage.getItem(CIRKEL_SLEUTEL) === 'list') ? 'reader' : 'grid';
+        body.dataset.feedView = 'grid';   // de enige mogelijkheid daar
         return;
       }
@@ -588 +590 @@
     var _feedC = ['on-home','on-tag','on-type','on-user','on-cirkel'];
     var _onFeed = _feedC.some(function(c){ return body.classList.contains(c); });
-    // Een weergave die op DEZE pagina niet bestaat, kan ook niet actief zijn.
-    // Lezen ontbreekt in de cirkel (daar staan berichten van anderen), en wie
-    // op de eigen feed Lezen koos houdt die voorkeur in localStorage. Zonder
-    // terugval staat de hele pil dan grijs terwijl de tijdlijn gewoon te zien
-    // is -- de lezer ziet nergens meer waar hij is. De VOORKEUR blijft staan:
-    // terug op de eigen feed is Lezen weer actief.
+    // Een weergave die op DEZE pagina niet KAN, kan ook niet actief zijn. In de
+    // cirkel staat Lezen er wel maar uitgeschakeld; pasWeergaveToe() heeft
+    // feedView daar al op 'grid' gezet, dus de terugval hieronder vuurt er niet.
+    // Hij blijft staan voor het geval een pagina een knop mist: zonder terugval
+    // staat de hele pil grijs terwijl er wel degelijk iets te zien is, en dan
+    // ziet de lezer nergens meer waar hij is. De solo-VOORKEUR blijft intact.
     var _view = String(body.dataset.feedView || 'reader');
     if (!/^[a-z]+$/.test(_view)) _view = 'reader';
     if (!document.querySelector('.view-switch-btn[data-view="' + _view + '"]')) _view = 'reader';
     document.querySelectorAll('.view-switch-btn').forEach(function(b) {
-      b.setAttribute('aria-selected', (_onFeed && b.dataset.view === _view) ? 'true' : 'false');
+      b.setAttribute('aria-selected', (_onFeed && !b.disabled && b.dataset.view === _view) ? 'true' : 'false');
     });
     document.querySelectorAll('.grid-cols-btn').forEach(function(b) {
@@ -635 +637 @@
     var sw = e.target.closest('.view-switch-btn');
     if (sw) {
+      // Een dode knop doet niets. Een <button disabled> vuurt in de meeste
+      // browsers geen click, maar deze listener hangt op document en vangt ook
+      // wat er langs de randen komt -- dus expliciet, niet op goed vertrouwen.
+      if (sw.disabled) return;
       var view = sw.dataset.view;
-      // EEN KEUZE MOET TERUG KUNNEN. Op de cirkel is Grid de enige knop (Lezen
-      // bestaat daar niet, want dat zijn berichten van anderen), en een knop die
-      // al actief is doet normaal niets -- dan zit je in Grid zonder weg terug.
-      // Is dit de enige knop op deze pagina en staat hij al aan, dan brengt hij
-      // je terug naar de standaard, en dat is daar de gewone lijst.
-      if (document.querySelectorAll('.view-switch-btn').length === 1
-          && sw.getAttribute('aria-selected') === 'true') {
-        view = 'reader';
-      }
       body.dataset.feedView = view;
-      // Op de cirkel onthouden we los, want daar betekent 'reader' de gewone
-      // lijst en hoort het je thuisweergave niet aan te raken.
+      // Op de cirkel schrijven we NIETS. Daar is Grid de enige mogelijkheid, dus
+      // er valt niets te onthouden -- en je solo-keuze mag er zeker niet door
+      // veranderen: die moet je terugvinden zoals je hem achterliet.
       try {
-        if (opCirkel()) localStorage.setItem(CIRKEL_SLEUTEL, view === 'grid' ? 'grid' : 'list');
-        else localStorage.setItem('pcms-feed-view', view);
+        if (!opCirkel()) localStorage.setItem('pcms-feed-view', view);
       } catch(_) {}
       syncAria();