Changeset 742ba7e in Klonkt for src/assets/js/guardian.js

Timestamp:

07/28/2026 07:58:26 PM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

6c152a5

Parents:

70677e96

Message:

Een ward loslaten vraagt eerst, en zegt wat het doet

Loslaten zat achter een window.confirm met een lap tekst erin. Dat is precies
het soort dialoog dat mensen wegklikken om ervan af te zijn, terwijl dit de
zwaarste knop in de app is. Nu is het een stap in het paneel: eerst de gevolgen,
dan ja of nee, met nee links zodat de uitweg de makkelijke is om te raken.

De waarschuwing wordt opgebouwd uit wat er echt aan de hand is, niet uit een
vaste zin, want loslaten betekent twee verschillende dingen. Blijven er andere
guardians over, dan stap jij op en blijft het kind een ward (FEP-633c 3.3). Ben
je de laatste, dan is het emancipatie, en 3.4 zegt uitdrukkelijk dat geen enkele
guardian daar alleen over gaat: drie instemmende volwassenen, of een meerderheid
met twee getuigen. Een nieuwe route zoekt dat op het moment van drukken op, want
voor een ward die wij niet hosten moet daarvoor die server bevraagd worden en
dat hoort niet bij elke ververs-ronde.

En het derde ding dat er hoorde te staan: het Undo federeert nog niet (fase 4 in
relations.js), dus vandaag haalt de knop de band alleen hier weg terwijl de
server van het kind je gewoon als guardian blijft noemen. Zonder die regel drukt
een guardian erop in de overtuiging dat het kind losgelaten is.

Changed files:
src/routes/guardian.js

• GET /guardian/wards/release-check: telt de guardians van dit kind (lokaal uit de relaties, remote uit shaer:guardians van de actor) en weigert een ward die niet van jou is
• de nieuwe labels toegevoegd aan uiStrings

src/assets/js/guardian.js

• releaseStep: de waarschuwing plus ja/nee, in plaats van window.confirm
• de site gaat mee in de check, anders kijkt de server naar de verkeerde

src/assets/css/guardian.css

• --danger toegevoegd; die bestond niet, waardoor het kader en de knop onzichtbaar bleven
• opmaak voor het waarschuwingsblok

src/services/i18n.js

• 24 strings voor de waarschuwing in nl, en, de

test/guardian-panel.test.js

• loslaten gaat in twee stappen en niet via window.confirm
• elke CSS-variabele die de PWA gebruikt is er ook een die hij definieert

remarks: alle vier de gevallen in de browser nagelopen: laatste guardian,
meerdere guardians, onbereikbare server, en een ward die niet van je is. Nee
zet de knop terug zonder iets te doen, ja verwijdert alleen die ene relatie.

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

File:

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

src/assets/js/guardian.js

@@ -184 +184 @@
   }
 
+  /**
+   * The second step of releasing a ward: what it does, then yes or no.
+   *
+   * The warning is assembled from what the server found, not from a fixed
+   * sentence, because releasing means two different things (FEP-633c): stepping
+   * down while other guardians remain (§3.3), or being the last one, which is
+   * emancipation and explicitly not one guardian's call (§3.4). And as long as
+   * the Undo does not federate, the ward's server keeps listing you either way
+   * — a guardian has to know that before pressing, not after.
+   */
+  function releaseStep(w, check, host, relBtn) {
+    var uri = w.other_uri;
+    var who = handleOf(uri, w.other_handle);
+    var box = el('div', 'g-warn');
+    box.appendChild(el('strong', null, (T.release_title || 'Release {who}?').replace('{who}', who)));
+    box.appendChild(el('p', null, T.release_effect || ''));
+    if (check.last === true) box.appendChild(el('p', 'grave', T.release_last || ''));
+    else if (check.last === false) box.appendChild(el('p', null, T.release_step_down || ''));
+    else box.appendChild(el('p', 'grave', T.release_unknown || ''));
+    if (check.federates === false) box.appendChild(el('p', null, T.release_local || ''));
+
+    var row = el('div', 'row');
+    var yes = el('button', 'danger small', T.release_yes || 'Yes');
+    yes.addEventListener('click', function () { yes.disabled = true; remove(uri, yes); });
+    var no = el('button', 'small', T.release_no || 'No');
+    no.addEventListener('click', function () {
+      host.removeChild(box);
+      relBtn.hidden = false; relBtn.disabled = false;
+    });
+    // No first: the way out should be the easy one to hit.
+    row.appendChild(no); row.appendChild(yes);
+    box.appendChild(row);
+    return box;
+  }
+
   function wardPanel(w) {
     var uri = w.other_uri;
@@ -215 +250 @@
     actRow.appendChild(wave);
     var rel = el('button', 'quiet small', T.release);
-    // Releasing a ward is heavy and hard to undo (coming back needs a fresh
-    // offer the ward accepts), so it asks first and spells out what changes.
+    // Letting a child go is a decision, not a click. It opens a step that first
+    // asks the server what releasing this particular ward actually does, then
+    // says it plainly and asks yes or no. Never window.confirm: that hides a
+    // long explanation behind an OK button people press to make it go away.
     rel.addEventListener('click', function () {
-      var who = handleOf(uri, w.other_handle);
-      var msg = (T.release_confirm || 'Release {who}?').replace('{who}', who);
-      if (window.confirm(msg)) remove(uri, rel);
+      rel.disabled = true;
+      // site matters: with several of your own sites the server would otherwise
+      // check this ward against the wrong one and answer "not my ward".
+      fetch('/guardian/wards/release-check?site=' + encodeURIComponent(S.site) + '&uri=' + encodeURIComponent(uri))
+        .then(function (r) { return r.json(); })
+        .then(function (c) {
+          rel.hidden = true;
+          act.appendChild(releaseStep(w, c || {}, act, rel));
+        })
+        .catch(function () { rel.disabled = false; });
     });
     actRow.appendChild(rel);