Skip to content

fix: make shared modal accessible - #1

Open
byot-dot wants to merge 1 commit into
SecretFoundation:masterfrom
byot-dot:fix/modal-accessibility
Open

fix: make shared modal accessible#1
byot-dot wants to merge 1 commit into
SecretFoundation:masterfrom
byot-dot:fix/modal-accessibility

Conversation

@byot-dot

@byot-dot byot-dot commented Sep 8, 2026

Copy link
Copy Markdown

Improves keyboard and assistive-technology support for the shared Modal component without changing its visual layout.

  • adds dialog semantics with role="dialog" and aria-modal
  • associates the dialog with its visible title and subtitle
  • gives the icon-only close button an accessible name
  • moves focus into the dialog when it opens and restores focus on close
  • contains Tab / Shift+Tab focus within the dialog
  • keeps Escape dismissal scoped to the active dialog

The change is limited to src/components/UI/Modal/Modal.tsx.

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