Skip to content

fix(ui): trap drawer keyboard focus#84

Merged
wyf027 merged 3 commits into
mainfrom
codex/drawer-focus-trap-2026-07-07
Jul 7, 2026
Merged

fix(ui): trap drawer keyboard focus#84
wyf027 merged 3 commits into
mainfrom
codex/drawer-focus-trap-2026-07-07

Conversation

@wyf027

@wyf027 wyf027 commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep Tab and Shift+Tab focus navigation inside an open Drawer
  • add Drawer keyboard focus trap coverage with user-event tab navigation
  • document Drawer focus looping and update the demo content with Tailwind layout classes

Validation

  • GitHub Actions CI / quality passed for head 3513e748ded6be12b6f643018bde7822f0d0542a.
  • Passed steps: npm run lint:ui, npm run typecheck:ui, npm run test:ui, npm run build:ui, npm run build:docs.
  • Test run included updated src/components/feedback/overlays/Drawer/Drawer.test.tsx coverage for Tab and Shift+Tab focus looping.
  • Not run locally: the Codex shell executor is currently failing with Failed to create unified exec process: No such file or directory.

Notes

@ecc-tools

ecc-tools Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

ECC bundle files are already tracked in this repository. Skipping generation of another bundle PR.

@wyf027
wyf027 merged commit 9a394a3 into main Jul 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant