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

[EuiResizableContainer][AXE-CORE]: Containers must allow keyboard users to access all content #5295

Closed
Tracked by #5265
1Copenut opened this issue Oct 20, 2021 · 4 comments · Fixed by #6534
Closed
Tracked by #5265

Comments

@1Copenut
Copy link
Contributor

Description

Sometimes our resizable containers have text content that does not receive keyboard focus. If we do not allow users to set focus on the container, they may not be able to read or access all information. I recommend we add a tabindex="0" on our EuiResizableContainer component. This could be a standard attribute, or added when content is "taller" than the container.

Guidance

Screenshots

Screen Shot 2021-10-20 at 2 56 55 PM

@1Copenut 1Copenut changed the title [EuiResizableContainer][AUTOMATION]: Containers must allow keyboard users to access all content, focusable or not [EuiResizableContainer][AUTOMATION]: Containers must allow keyboard users to access all content Oct 20, 2021
@1Copenut 1Copenut changed the title [EuiResizableContainer][AUTOMATION]: Containers must allow keyboard users to access all content [EuiResizableContainer][AXE-CORE]: Containers must allow keyboard users to access all content Oct 20, 2021
@github-actions
Copy link

👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

@1Copenut
Copy link
Contributor Author

Keep this issue open please.

@github-actions
Copy link

👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

@github-actions
Copy link

❌ We're automatically closing this issue due to lack of activity. Please comment if you feel this was done in error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants