Skip to content

Conversation

@reidbarber
Copy link
Member

@reidbarber reidbarber commented Nov 17, 2025

  • Fixes Unexpected duplicate view-transition-name error that was thrown when closing the search menu
  • Hides the old elements while the new ones are transitioning in

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

Open and close the search menu is various browsers.

🧢 Your Project:

@reidbarber reidbarber changed the title fix(S2 docs): search menu view transitions fix: S2 docs search menu view transitions Nov 17, 2025
@rspbot
Copy link

rspbot commented Nov 17, 2025

LFDanLu
LFDanLu previously approved these changes Nov 17, 2025
Copy link
Member

@LFDanLu LFDanLu left a comment

Choose a reason for hiding this comment

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

LGTM, I just noticed the builds have been failing so prob best to get a up to date build link for this PR before it goes in so we can test off that

@rspbot
Copy link

rspbot commented Nov 17, 2025

LFDanLu
LFDanLu previously approved these changes Nov 18, 2025
@reidbarber reidbarber changed the title fix: S2 docs search menu view transitions fix(docs): S2 docs search menu view transitions Nov 18, 2025
devongovett
devongovett previously approved these changes Nov 18, 2025
@reidbarber reidbarber added this pull request to the merge queue Nov 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Nov 18, 2025
@devongovett devongovett dismissed stale reviews from LFDanLu and themself via fd40b3b November 18, 2025 01:36
@rspbot
Copy link

rspbot commented Nov 18, 2025

@devongovett devongovett merged commit 2971aae into main Nov 18, 2025
32 checks passed
@devongovett devongovett deleted the s2-docs-view-transition-fixes branch November 18, 2025 01:52
bsmiley1 pushed a commit to bsmiley1/react-spectrum that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants