Skip to content

fix: Safari FocusScope contain bug with native button#9714

Draft
snowystinger wants to merge 3 commits intomainfrom
focusscope-safari-button-bug
Draft

fix: Safari FocusScope contain bug with native button#9714
snowystinger wants to merge 3 commits intomainfrom
focusscope-safari-button-bug

Conversation

@snowystinger
Copy link
Member

Closes #9570

note, if focus isn't in the input in the example in the S2 storybook example App, click to put focus there so that the FocusScope is activated.

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

@rspbot
Copy link

rspbot commented Mar 1, 2026

@snowystinger
Copy link
Member Author

Do we want to do this? there's a way to work around the original issue, which is to put a tab index on your buttons. This may not always be possible though if another library is supplying the button.

@rspbot
Copy link

rspbot commented Mar 16, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FocusScope with contain prevents clicking buttons in scrollable content

2 participants