Skip to content

docs: fix search menu styles#9575

Merged
LFDanLu merged 10 commits intomainfrom
fix-search-menu-styles
Feb 3, 2026
Merged

docs: fix search menu styles#9575
LFDanLu merged 10 commits intomainfrom
fix-search-menu-styles

Conversation

@reidbarber
Copy link
Member

@reidbarber reidbarber commented Feb 2, 2026

Pulled changes from #9524 to get into this release:

  • Add horizontal scrolling to search tags to avoid overflow (same as we already do for mobile)
  • Start-align the info messages in Icons and Color search.
  • Make the Icons and Color search layout styles consistent
  • Trim icon/illustration search values

✅ 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:

  1. On desktop, resize the window so that the search menu shrinks and verify that the search tags have horizontal scrolling
  2. Verify that the Icons and Colors search info messages are both start-aligned instead of center-aligned.
  3. Test Colors search on mobile, the listbox shouldn't have any horizontal scrolling.
  4. Check that icons/illustrations search does not return an empty list when searching whitespace

🧢 Your Project:

snowystinger
snowystinger previously approved these changes Feb 2, 2026
@rspbot
Copy link

rspbot commented Feb 2, 2026

@yihuiliao
Copy link
Member

Screenshot 2026-02-02 at 4 42 39 PM

Can we add align-items: center to the div?

@rspbot
Copy link

rspbot commented Feb 3, 2026

@LFDanLu LFDanLu added this pull request to the merge queue Feb 3, 2026
Merged via the queue into main with commit 00f9060 Feb 3, 2026
31 checks passed
@LFDanLu LFDanLu deleted the fix-search-menu-styles branch February 3, 2026 18:11
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.

5 participants