From bf199225630e5238fba23bee76f97d0e1eba8937 Mon Sep 17 00:00:00 2001 From: Vince Castillo Date: Sat, 4 Jul 2026 09:06:40 -0700 Subject: [PATCH] fix: expose ConfirmDialog with dialog role and label its input (WCAG 4.1.2, 3.3.2) ConfirmDialog trapped focus and closed on Escape but its container was a plain div, so screen readers did not announce it as a modal dialog. Its text input also had only a placeholder, giving no persistent accessible name. Add role=dialog / aria-modal / aria-label / tabindex to the dialog surface and an aria-label to the textarea. Relates to #2790 --- src/lib/components/common/ConfirmDialog.svelte | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/lib/components/common/ConfirmDialog.svelte b/src/lib/components/common/ConfirmDialog.svelte index 10b5037414..e5737b9614 100644 --- a/src/lib/components/common/ConfirmDialog.svelte +++ b/src/lib/components/common/ConfirmDialog.svelte @@ -116,6 +116,10 @@ }} >