Skip to content

Make account analysis modal accessible - #152

Merged
Justice989810 merged 1 commit into
Justice989810:mainfrom
valentinachristopher911-design:fix/issue-91-accessible-analysis-dialog
Sep 29, 2026
Merged

Justice989810 merged 1 commit into
Justice989810:mainfrom
valentinachristopher911-design:fix/issue-91-accessible-analysis-dialog

Conversation

@valentinachristopher911-design

Copy link
Copy Markdown

Summary

  • Use a native modal dialog with an accessible title and browser-managed focus containment.
  • Focus the close button when opened, close on Escape, and restore focus to the opener when dismissed.

Verification

  • npm run build passes.
  • npx eslint src/components/AccountAnalysisModal.tsx reports the pre-existing explicit any types.

Fixes #91

@Justice989810
Justice989810 merged commit f786beb into Justice989810:main Sep 29, 2026
1 check failed
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.

Give the account analysis modal dialog semantics and focus management

2 participants