Settings: the server configuration editor traps Tab (WCAG 2.1.2) #132

Closed
opened 2026-09-25 22:17:40 +00:00 by kayg · 1 comment
Owner

Severity: medium (accessibility: WCAG 2.1.2 No Keyboard Trap)

Problem

Settings → Admin → Server configuration: the TOML editor's comment says "Escape then Tab leaves", but Tab always inserted two spaces. A keyboard user who Tabs into the editor cannot Tab forward past it (the break-it keyboard walk cycles between "Copy link" and the editor at 390 and 1440).

Repro

Open /settings/admin/configuration, focus the editor, press Tab.

Expected

Escape, then Tab moves focus on (the CodeMirror rule). A second Escape closes Settings.

Status

Fixed on job/breakit-fixes in 4ac3ff0 (checked in the real app: Esc+Tab lands on "Save configuration"; Settings stays open). Not merged. Consider a visible hint under the editor ("Esc then Tab to leave").

Found by the break-it sweep (#117). Re-run: cd apps/web && bun run build && bun e2e/breakit.mjs --keep <dir> (script on branch job/breakit-fixes). Screenshots: /home/kayg/Developer/calternal/target/breakit (run 1 in run1/, fix checks in verify/).

**Severity:** medium (accessibility: WCAG 2.1.2 No Keyboard Trap) ## Problem Settings → Admin → Server configuration: the TOML editor's comment says "Escape then Tab leaves", but Tab always inserted two spaces. A keyboard user who Tabs into the editor cannot Tab forward past it (the break-it keyboard walk cycles between "Copy link" and the editor at 390 and 1440). ## Repro Open `/settings/admin/configuration`, focus the editor, press Tab. ## Expected Escape, then Tab moves focus on (the CodeMirror rule). A second Escape closes Settings. ## Status Fixed on `job/breakit-fixes` in 4ac3ff0 (checked in the real app: Esc+Tab lands on "Save configuration"; Settings stays open). Not merged. Consider a visible hint under the editor ("Esc then Tab to leave"). Found by the break-it sweep (#117). Re-run: `cd apps/web && bun run build && bun e2e/breakit.mjs --keep <dir>` (script on branch `job/breakit-fixes`). Screenshots: `/home/kayg/Developer/calternal/target/breakit` (run 1 in `run1/`, fix checks in `verify/`).
Author
Owner

Merged into dev (ui-batch 642dfafc / upload-scale b74e772c / tz-days db733d1). Deploy status is tracked on #203.

Merged into dev (ui-batch 642dfafc / upload-scale b74e772c / tz-days db733d1). Deploy status is tracked on #203.
kayg closed this issue 2026-09-26 16:00:01 +00:00
Sign in to join this conversation.
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
kayg/calternal#132
No description provided.