Skip to content

[7.x] [Security Solution] Accessibility (a11y) fixes (#87783)#87970

Merged
andrew-goldstein merged 2 commits intoelastic:7.xfrom
andrew-goldstein:backport/7.x/pr-87783
Jan 12, 2021
Merged

[7.x] [Security Solution] Accessibility (a11y) fixes (#87783)#87970
andrew-goldstein merged 2 commits intoelastic:7.xfrom
andrew-goldstein:backport/7.x/pr-87783

Conversation

@andrew-goldstein
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

## [Security Solution] Accessibility (a11y) fixes

This PR fixes the following accessibility (a11y) issues:

- Fixes an issue that prevented tabbing through all elements on pages with embedded Timelines
- Fixes an issue where the Timeline data providers popover menu was not displayed when Enter is pressed
- Fixes an issue where duplicate draggable IDs caused errors when re-arranging Timeline columns
- Fixes an issue where Timeline columns could not be removed or sorted via keyboard
- Fixes an issue where focus is not restored to the `Customize Columns` button when the `Reset` button is pressed
- Fixes an issue where filtering the `Customize Event Renderers` view via the input cleared selected entries
- Fixes an issue where the active timeline button wasn't focused when Timeline is closed
- Fixes an issue where the `(+)` Create / Open Timeline button's hover panel didn't own focus
@andrew-goldstein andrew-goldstein added the backport This PR is a backport of another PR label Jan 12, 2021
@andrew-goldstein
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 8.5MB 8.5MB +9.8KB

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@andrew-goldstein andrew-goldstein merged commit 97d8ef1 into elastic:7.x Jan 12, 2021
@andrew-goldstein andrew-goldstein deleted the backport/7.x/pr-87783 branch January 12, 2021 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants