-
Notifications
You must be signed in to change notification settings - Fork 841
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
[EuiSelectable] Sometimes needs double click when used inside EuiPopover #4147
Comments
This is similar to #4128 and has do with the fact that the list is holding on to the focus of that first item. So when the list is then clicked initially it's just setting the focus on the list not the highlighted item. I think this may be a regression from the update to virtualized-window. @myasonik Can you take a look? It's reproducible/happens in the Searchable example. https://elastic.github.io/eui/#/forms/selectable#searchable |
It looks like this wasn't completely fixed bug_1.mov
cc @myasonik |
Looks like this is still an issue, and doesn't actually have anything to do with EuiPopover. Can be reproduced in the docs with a single selection EuiSelectable: Kapture.2021-08-31.at.09.59.54.mp4Something with initial focus logic prevents the first click from action. |
When using
EuiSelectable
insideEuiPopover
, sometimes clicking to make a new selection doesn't work at the first click and the user needs to click a second time for the selection to happen. On the first click the focus goes to the currently selected item instead. This happens when there is scrolling occurring insideEuiSelectable
.This is currently happening in Discover and Lens as reported here
The text was updated successfully, but these errors were encountered: