Skip to content

Fix AlertDialog initial focus and dismiss affordance - #591

Merged
czarandy merged 2 commits into
mainfrom
fix/547-alert-dialog-focus
Aug 25, 2026
Merged

Fix AlertDialog initial focus and dismiss affordance#591
czarandy merged 2 commits into
mainfrom
fix/547-alert-dialog-focus

Conversation

@czarandy

Copy link
Copy Markdown
Owner

Closes #547

Summary

  • focus the AlertDialog cancel action when the dialog opens
  • suppress the automatic LayoutHeader close button in AlertDialog
  • add reusable Button autofocus and LayoutHeader close-button controls
  • cover the behavior with unit tests and a Storybook example

Verification

  • pnpm test
  • pnpm lint
  • pnpm format:check
  • pnpm typecheck
  • pnpm build
  • pnpm build:storybook

@vercel

vercel Bot commented Aug 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
silver-ui-site Ready Ready Preview Aug 25, 2026 8:49pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
silver-ui Ignored Ignored Preview Aug 25, 2026 8:49pm

@czarandy
czarandy merged commit d9a7045 into main Aug 25, 2026
4 checks passed
@czarandy
czarandy deleted the fix/547-alert-dialog-focus branch August 25, 2026 23:25
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.

AlertDialog: initial focus lands on the title heading instead of the least-destructive action

1 participant