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

Avatar component not focusable with keyboard #613

Closed
georgehrke opened this issue Sep 30, 2019 · 3 comments · Fixed by #1854
Closed

Avatar component not focusable with keyboard #613

georgehrke opened this issue Sep 30, 2019 · 3 comments · Fixed by #1854
Assignees
Labels
1. to develop Accepted and waiting to be taken care of accessibility Making sure we design for the widest range of people possible, including those who have disabilities bug Something isn't working feature: avatar Related to the avatar component

Comments

@georgehrke
Copy link
Contributor

Meaning that the contacts-menu is not also not accessible.

@georgehrke georgehrke added bug Something isn't working 1. to develop Accepted and waiting to be taken care of feature: avatar Related to the avatar component accessibility Making sure we design for the widest range of people possible, including those who have disabilities labels Sep 30, 2019
@skjnldsv
Copy link
Contributor

:fixit: :D

@PVince81
Copy link
Contributor

PVince81 commented Jan 19, 2021

ideally we should move to the Actions component as it has everything needed for keyboard focus and moving through the items with arrows: #1670

@PVince81
Copy link
Contributor

WIP PR here: #1854

but not sure if it can be finishable as the interaction in the popover is missing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of accessibility Making sure we design for the widest range of people possible, including those who have disabilities bug Something isn't working feature: avatar Related to the avatar component
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants