Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

AR-2135 Fix Select closing when interactive children receive focus #327

Merged
merged 2 commits into from
Mar 8, 2021

Conversation

justinanastos
Copy link
Contributor

@justinanastos justinanastos commented Mar 5, 2021

When an interactive child of Select is focused, like an <input />; useSelect will recognize that as a menu blur and will close the menu. This will capture the menu blur and mouse leave events and keep the Select open if the activeElement (element that has the focus) is a descendent of the underlying tooltip.

AR-2135

📦 Published PR as canary version: 8.17.3-canary.327.8663.0

✨ Test out this PR locally via:

npm install @apollo/space-kit@8.17.3-canary.327.8663.0
# or 
yarn add @apollo/space-kit@8.17.3-canary.327.8663.0

@justinanastos justinanastos added the bug fix Increment the patch version when merged label Mar 5, 2021
@justinanastos justinanastos changed the title Fix Select closing when interactive children receive focus AR-2135 Fix Select closing when interactive children receive focus Mar 5, 2021
@justinanastos justinanastos requested review from jgzuke and a team March 5, 2021 20:40
@justinanastos justinanastos marked this pull request as ready for review March 5, 2021 20:40
@justinanastos justinanastos merged commit 0c13cff into main Mar 8, 2021
@justinanastos justinanastos deleted the justin/AR-2135/select-children branch March 8, 2021 16:11
@apollo-bot2
Copy link
Collaborator

🚀 PR was released in v8.17.3 🚀

@apollo-bot2 apollo-bot2 added the released This issue/pull request has been released. label Mar 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug fix Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants