Skip to content

fix(@react-stately/combobox): don't commit selection on close #3912

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jan 18, 2023

Conversation

alirezamirian
Copy link
Contributor

@alirezamirian alirezamirian commented Jan 12, 2023

Regression introduced in 168ca5a.

Closes #3900

✅ Pull Request Checklist:

  • Included link to the 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:

The code sample here can be used to test: https://react-spectrum.adobe.com/react-aria/usePopover.html#example
Also, related potentially affected components would be Combobox and SearchAutocompletion

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, did some more testing and haven't found any problematic cases. For other reviewers, please refer to the discussion in #3900 for more details as to why this change is being made.

@snowystinger
Copy link
Member

GET_BUILD

@rspbot
Copy link

rspbot commented Jan 18, 2023

@adobe adobe deleted a comment from rspbot Jan 18, 2023
Copy link
Member

@snowystinger snowystinger left a comment

Choose a reason for hiding this comment

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

Did not find any issues in our storybook. Looks good to me.

@LFDanLu LFDanLu merged commit 93f49a4 into adobe:main Jan 18, 2023
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.

Clicking outside combobox selects the last focused item
4 participants