Skip to content

Conversation

yihuiliao
Copy link
Member

@yihuiliao yihuiliao commented Oct 3, 2025

  • updates expandedKey in S2 accordion docs
  • increase gap between icons under "Available Icons" so focus ring doesn't get cut off
  • fix focus ring cutting through text on Tabs in SearchMenu

✅ 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 Oct 3, 2025


function Example() {
let [expandedKeys, setExpandedKeys] = useState(new Set<Key>(['system']));
let [expandedKeys, setExpandedKeys] = useState(new Set<Key>(['settings']));
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe leftover from a copy paste but noticed that our example didn't have key with "system" so updated it

@LFDanLu LFDanLu added this pull request to the merge queue Oct 6, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 6, 2025
@LFDanLu LFDanLu added this pull request to the merge queue Oct 6, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 6, 2025
@LFDanLu LFDanLu added this pull request to the merge queue Oct 6, 2025
@LFDanLu
Copy link
Member

LFDanLu commented Oct 6, 2025

not sure why it keeps failing on the react 17 tests, will have to look into that test separately

Merged via the queue into main with commit 39e6ade Oct 6, 2025
33 checks passed
@LFDanLu LFDanLu deleted the s2-doc-quick-fix branch October 6, 2025 18:24
LFDanLu pushed a commit that referenced this pull request Oct 9, 2025
* docs: update the expanded key in s2 accordion docs

* increase gap between icons in available icons

* fix focus ring on search menu tabs
devongovett pushed a commit that referenced this pull request Oct 9, 2025
* docs: update the expanded key in s2 accordion docs

* increase gap between icons in available icons

* fix focus ring on search menu tabs
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.

4 participants