Changeset 742ba7e in Klonkt for test

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:

• test/guardian-panel.test.js (modified) (1 diff)

test/guardian-panel.test.js

@@ -39 +39 @@
 });
 
+test('releasing a ward is two steps, not a browser confirm', () => {
+  // Robins besluit: letting a child go is a decision. window.confirm hides a
+  // long explanation behind an OK button people press to make it go away.
+  // A call, not the word: the comment above the release button names it too.
+  assert.ok(!/window\.confirm\s*\(/.test(client), 'the release must ask in the panel, with its own yes and no');
+  assert.match(client, /release-check/, 'and it first asks the server what releasing this ward actually does');
+  for (const k of ['release_last', 'release_step_down', 'release_unknown']) {
+    assert.ok(client.includes(k), `the warning must cover the ${k} case`);
+  }
+  assert.match(route, /not_my_ward/, 'and the check refuses a ward that is not yours');
+});
+
+test('every CSS variable the PWA uses is one it defines', () => {
+  // The dashboard is standalone: it never inherits the site theme, so an
+  // undefined var silently renders as nothing. --danger did exactly that: the
+  // warning box lost its border and the confirm button its background.
+  const css = read('../src/assets/css/guardian.css');
+  const root = (css.match(/:root\s*\{([\s\S]*?)\}/) || [])[1] || '';
+  const defined = new Set([...root.matchAll(/(--[a-z0-9-]+)\s*:/g)].map((m) => m[1]));
+  const used = [...new Set([...css.matchAll(/var\((--[a-z0-9-]+)(\s*,)?/g)]
+    .filter((m) => !m[2])            // one with a fallback is fine
+    .map((m) => m[1]))];
+  const missing = used.filter((v) => !defined.has(v));
+  assert.deepEqual(missing, [], `guardian.css uses ${missing.join(', ')} without defining it`);
+});
+
 test('the labels the panel renders are all served to it', () => {
   // uiStrings() picks the keys by hand, so a label used in the client but not