Skip to content

Fix Popover focus restoration - #444

Merged
czarandy merged 1 commit into
mainfrom
fix/issue-201-popover-focus
Aug 7, 2026
Merged

Fix Popover focus restoration#444
czarandy merged 1 commit into
mainfrom
fix/issue-201-popover-focus

Conversation

@czarandy

@czarandy czarandy commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Restore focus to the previously focused element after Popover autofocus.
  • Cancel pending autofocus on close and leave focus unchanged when autofocus never moved it.
  • Add regression coverage for Escape dismissal and no-focus light-dismiss behavior.

Closes #201

Testing

  • pnpm test (148 files, 3,200 tests)
  • pnpm typecheck
  • pnpm lint
  • pnpm build

@vercel

vercel Bot commented Aug 7, 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 7, 2026 6:44pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
silver-ui Ignored Ignored Preview Aug 7, 2026 6:44pm

@czarandy
czarandy enabled auto-merge (squash) August 7, 2026 18:45
@czarandy
czarandy merged commit 4516018 into main Aug 7, 2026
4 checks passed
@czarandy
czarandy deleted the fix/issue-201-popover-focus branch August 7, 2026 18:48
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.

Popover doesn't return focus to the trigger on close

1 participant