Replace native confirm() dialogs with Cairn modals (revoke share, delete) #196

New issue

Closed

opened 2026-08-11 22:09:07 +00:00 by Cordy · 0 comments

Cordy commented

2026-08-11 22:09:07 +00:00

Owner

Dogfood (Nikola): destructive confirmations use the browser's built-in confirm() dropdown, which looks foreign to the app:

Revoking a share — confirm(tf("revokeShareConfirm", …)) in openSharesView (index.html).
Deleting a folder (and file) — confirm(…) in the delete handler.

These should use Cairn's own styled modal, consistent with the rest of the UI (the app already has a <dialog> + showModal() pattern — e.g. the Share dialog — to model after).

Scope: sweep every confirm() (and any prompt(), e.g. rename) in web/static/index.html and route them through a small reusable Cairn confirm/prompt modal (title, message, Cancel / destructive-action buttons; Enter/Escape handling; focus trap). Keep the i18n strings; just change the presentation.

Frontend polish — batched for a UI pass (not urgent, but visible on every destructive action).

Dogfood (Nikola): destructive confirmations use the browser's built-in `confirm()` dropdown, which looks foreign to the app: - **Revoking a share** — `confirm(tf("revokeShareConfirm", …))` in `openSharesView` (index.html). - **Deleting a folder** (and file) — `confirm(…)` in the delete handler. These should use Cairn's own styled modal, consistent with the rest of the UI (the app already has a `<dialog>` + `showModal()` pattern — e.g. the Share dialog — to model after). Scope: sweep every `confirm()` (and any `prompt()`, e.g. rename) in `web/static/index.html` and route them through a small reusable Cairn confirm/prompt modal (title, message, Cancel / destructive-action buttons; Enter/Escape handling; focus trap). Keep the i18n strings; just change the presentation. Frontend polish — batched for a UI pass (not urgent, but visible on every destructive action).