Changeset 1ff801d in Klonkt for src/assets/js/guardian.js

Timestamp:

08/07/2026 03:53:57 PM (5 weeks ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

0d5bd2c

Parents:

718eaeb

git-author:

Robin <roboburr@…> (08/07/2026 03:53:55 PM)

git-committer:

roboburr <roboburr@…> (08/07/2026 03:53:57 PM)

Message:

De hulpboei op de wardregel telt alleen wat nog wacht (shaer-lgo, vervolg)

Barts melding: na het afhandelen bleef er een 🛟 1 op de wardregel staan.

Dezelfde fout als de vorige, op een tweede plek. De teller op die regel telde
alle hulpvragen van dat kind, ook afgehandelde -- terwijl het commentaar er direct
boven al zei "whatever is WAITING must be visible with the panel shut". De
bedoeling klopte, de uitvoering niet.

DE ECHTE LES: "open" werd op twee plekken los beslist. Ik filterde de lijst en
dacht dat ik klaar was. Nu is er een helpOpen(h), en die is de enige plek waar
dat wordt uitgemaakt -- met in het commentaar waarom, zodat een derde plek er
langskomt in plaats van het opnieuw te bedenken.

Bij twijfel telt hij OPEN: alles wat geen expliciete afsluiting draagt wacht nog.
De omgekeerde fout -- iets als afgehandeld tonen dat het niet is -- is hier de
gevaarlijke.

Ook: in het paneel van een kind blijft de VOLLEDIGE geschiedenis staan (daar is
dat paneel voor), maar staat wat nog wacht bovenaan. Dat is waar je naar zoekt
als je het openslaat.

Suite 576/576, al raakt die dit niet: client-JS.

File:

• src/assets/js/guardian.js (modified) (4 diffs)

src/assets/js/guardian.js

@@ -49 +49 @@
   // child's panel.
   var HELP_TOP = 5;
+
+  /**
+   * Wacht deze hulpvraag nog op iemand?
+   *
+   * EEN plek, met opzet. Dit werd eerst op twee plekken los beslist -- de lijst
+   * en de teller op de wardregel -- en toen de lijst gefilterd werd bleef die
+   * teller een afgehandelde vraag meetellen. Een derde plek komt er vast, en dan
+   * hoort hij hier langs te komen.
+   *
+   * Bij TWIJFEL open: alles wat geen expliciete afsluiting draagt telt als
+   * wachtend. De omgekeerde fout -- iets als afgehandeld tonen dat het niet is --
+   * is hier de gevaarlijke.
+   */
+  function helpOpen(h) { return !h || !h.state || h.state.open; }
 
   function helpCard(h) {
@@ -157 +171 @@
     // meer in de teller: anders blijft het cijfer alarm slaan voor iets dat af
     // is, en neemt een gesloten vraag de ruimte in van een open vraag.
-    var open = alle.filter(function (h) { return !h.state || h.state.open; });
-    var klaar = alle.filter(function (h) { return h.state && !h.state.open; });
+    var open = alle.filter(helpOpen);
+    var klaar = alle.filter(function (h) { return !helpOpen(h); });
 
     open.slice(0, HELP_TOP).forEach(function (h) { list.appendChild(helpCard(h)); });
@@ -580 +594 @@
       T.panel_follow_empty || '', followCard);
 
+    // Hier juist de VOLLEDIGE geschiedenis van dit kind -- dat is waar dit
+    // paneel voor is. Wel wat nog wacht bovenaan, want dat is waar je naar zoekt
+    // als je het opent.
     sectionInto(panel, T.panel_help || 'Calls for help',
-      (S.help || []).filter(function (h) { return h.actor_uri === uri; }),
+      (S.help || []).filter(function (h) { return h.actor_uri === uri; })
+        .sort(function (a, b) { return (helpOpen(b) ? 1 : 0) - (helpOpen(a) ? 1 : 0); }),
       T.panel_help_empty || '', helpCard);
 
@@ -631 +649 @@
       row.appendChild(el('span', 'who grow', handleOf(uri, w.other_handle)));
       // Counts on the row: whatever is waiting must be visible with the panel shut.
-      var nHelp = (S.help || []).filter(function (h) { return h.actor_uri === uri; }).length;
+      // Alleen wat nog WACHT, zoals het commentaar hierboven al zei: een
+      // afgehandelde vraag hoorde de boei niet te laten staan.
+      var nHelp = (S.help || []).filter(function (h) { return h.actor_uri === uri && helpOpen(h); }).length;
       var nFollow = FOLLOWS.filter(function (f) { return f.wardUri === uri; }).length;
       if (nHelp) row.appendChild(el('span', 'tag help', '🛟 ' + nHelp));