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

Do not add aria-label when avatar is not interactive #3618

Merged
merged 1 commit into from
Jan 6, 2023

Conversation

Pytal
Copy link
Contributor

@Pytal Pytal commented Jan 6, 2023

aria-label should only be present when the element is interactive

Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal Pytal added bug Something isn't working 3. to review Waiting for reviews 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 Jan 6, 2023
@Pytal Pytal added this to the 7.3.1 milestone Jan 6, 2023
@Pytal Pytal self-assigned this Jan 6, 2023
@Pytal Pytal mentioned this pull request Jan 6, 2023
11 tasks
@skjnldsv skjnldsv merged commit 89b7437 into master Jan 6, 2023
@skjnldsv skjnldsv deleted the fix/a11y-non-interactive-avatar branch January 6, 2023 10:06
@skjnldsv skjnldsv mentioned this pull request Jan 11, 2023
@skjnldsv skjnldsv modified the milestones: 7.3.1, 7.4.0 Jan 11, 2023
@skjnldsv skjnldsv mentioned this pull request Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews 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 this pull request may close these issues.

None yet

3 participants