Skip to content
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

[Unified Search] Data View Menu: Keyboard focus issues #131566

Closed
cchaos opened this issue May 4, 2022 · 5 comments · Fixed by #132451
Closed

[Unified Search] Data View Menu: Keyboard focus issues #131566

cchaos opened this issue May 4, 2022 · 5 comments · Fixed by #132451
Labels
accessibility: focus Accessibility focus management accessibility: keyboard navigation Accessibility keyboard navigation and focus blocked Feature:Unified search Unified search related tasks Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.3.0

Comments

@cchaos
Copy link
Contributor

cchaos commented May 4, 2022

2.1. When choosing the option that closes the popover via keyboard the focus is lost

a11ypopover.mp4
  1. Edit filter popover: Same here - before it was working by focusing the next element, now it looses focus (screenshot on state on main) after editing an existing filter

Screenshot 2022-05-03 at 21 47 08

Originally posted by @mbondyra in #128401 (comment)

@botelastic botelastic bot added the needs-team Issues missing a team label label May 4, 2022
@cchaos cchaos added accessibility: focus Accessibility focus management Feature:Unified search Unified search related tasks accessibility: keyboard navigation Accessibility keyboard navigation and focus v8.3.0 labels May 4, 2022
@cchaos
Copy link
Contributor Author

cchaos commented May 4, 2022

@mbondyra & @stratoula I investigated the Data View menu return focus issue by narrowing it down to EuiSelectable within an EuIContextMenuPanel. I looked at how Lens was doing this in https://github.com/stratoula/kibana/blob/3d8c2551f0c3186a935e93c0fc197cecb5c311f7/x-pack/plugins/lens/public/indexpattern_datasource/change_indexpattern.tsx which works, and then wrapped the <DataViewsList> in an <EuiContextMenuPanel> and focus now gets lost after selection.

I'm not terribly familiar with programatic focus-management, so I can't tell if this is specifically an EUI issue or if we just need to manually handle this situation by adding a ref on the trigger button and forcing focus on index-pattern change.

I'm cc-ing @elastic/eui-design in case this sounds familiar to them.

@cchaos
Copy link
Contributor Author

cchaos commented May 4, 2022

For no. 2 (the filter add button), I can't replicate the focus issues.

✅ When I open the popover and close it immediately, focus is returned.
✅ When I open the popover and add the filter, focus is returned to the add button.

@mbondyra
Copy link
Contributor

mbondyra commented May 4, 2022

@cchaos About the no. 2 - it doesn't happen for add the filter, it happens for when you edit the existing filter pill. It also happens for filter popover and data view menu. So basically all elements inside EuiContextMenu. I can also reproduce it in EUI. i've opened this issue: elastic/eui#5869

@cchaos
Copy link
Contributor Author

cchaos commented May 4, 2022

Thank you @mbondyra !

@stratoula stratoula added blocked Team:Visualizations Visualization editors, elastic-charts and infrastructure labels May 5, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors)

@botelastic botelastic bot removed the needs-team Issues missing a team label label May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility: focus Accessibility focus management accessibility: keyboard navigation Accessibility keyboard navigation and focus blocked Feature:Unified search Unified search related tasks Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.3.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants