Dialog canon completed - fact-row confirms, structure, dead trio removed (v0.6.258) #707

Merged
Cordy merged 10 commits from fix-693 into main 2026-09-23 22:58:42 +00:00
Owner

Closes #693

The shared #dangerdlg generalizes spacedeldlg's fact-row canon (.conseq/.crow/.cico/.ctext, now shared selectors): title, icon + sentence rows with bold counts and mono paths, Cancel before a named danger verb. All eight destructive confirms migrate: file delete, bulk delete, link revoke ×2, app-password revoke, delete-forever (with warn row), peer remove, peer-key revoke. askConfirm remains only for the trivial peer enable/disable toggle, per canon.

Structure: peerdetaildlg gains an h2 (peerDetailTitle ×4); peeraddlg/peerdetaildlg/spcdetaildlg swap actions dlgfoot for .dlgactions. #pickdlg is documented as a style-guide exception (its composite footer carries the destination readout; New-folder is a tree action).

Dead trio: settingsdlg/auditdlg/holdsdlg become hidden parking-lot divs — their children are live admin-shell widgets adopted at open (SET_SLOTS / audit + holds slots), so only the dialog chrome and boot wiring (openSettings, set-close/set-save/set-audit/set-holds, audit-close, holds-close) are removed. saveSettings, ensureSettingsInShell and the adm-save-* buttons are untouched.

New i18n ×4: dgRevokeLink, dgRevokePw, dgRemovePeer, dgRevokeKey, dgNoteTrash, dgNoteLink, dgNotePurge, peerDetailTitle.

Pins in web/test/qa693.test.js; qa7.test.js caller-count pins superseded there.

Closes #693 The shared `#dangerdlg` generalizes spacedeldlg's fact-row canon (`.conseq`/`.crow`/`.cico`/`.ctext`, now shared selectors): title, icon + sentence rows with bold counts and mono paths, Cancel before a named danger verb. All eight destructive confirms migrate: file delete, bulk delete, link revoke ×2, app-password revoke, delete-forever (with warn row), peer remove, peer-key revoke. `askConfirm` remains only for the trivial peer enable/disable toggle, per canon. Structure: peerdetaildlg gains an h2 (`peerDetailTitle` ×4); peeraddlg/peerdetaildlg/spcdetaildlg swap `actions dlgfoot` for `.dlgactions`. `#pickdlg` is documented as a style-guide exception (its composite footer carries the destination readout; New-folder is a tree action). Dead trio: `settingsdlg`/`auditdlg`/`holdsdlg` become hidden parking-lot divs — their children are live admin-shell widgets adopted at open (SET_SLOTS / audit + holds slots), so only the dialog chrome and boot wiring (openSettings, set-close/set-save/set-audit/set-holds, audit-close, holds-close) are removed. `saveSettings`, `ensureSettingsInShell` and the `adm-save-*` buttons are untouched. New i18n ×4: dgRevokeLink, dgRevokePw, dgRemovePeer, dgRevokeKey, dgNoteTrash, dgNoteLink, dgNotePurge, peerDetailTitle. Pins in web/test/qa693.test.js; qa7.test.js caller-count pins superseded there.
A shared dangerdlg brings the fact-row canon (icon + sentence with bold counts and mono paths, Cancel before a named danger verb) to every destructive confirm: file and bulk delete, link and app-password revoke, purge, peer remove and key revoke - eight callers leave askConfirm, which keeps only the trivial enable/disable toggle. peerdetaildlg gains a title; the three detail/add dialogs get real .dlgactions bands; pickdlg's composite footer is documented as a style-guide exception. The unreachable settings/audit/holds dialogs become hidden parking-lot divs (their children are live admin-shell widgets moved out at open) and their boot wiring - openSettings, set-close/set-save/set-audit/set-holds, audit-close, holds-close - is removed.
changelog: v0.6.258
All checks were successful
ci / test-and-build (pull_request) Successful in 1m16s
2754db7e50
Cordy merged commit 29ce3b3ede into main 2026-09-23 22:58:42 +00:00
Cordy deleted branch fix-693 2026-09-23 22:58:44 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Cordy/Cairn#707
No description provided.