-
Notifications
You must be signed in to change notification settings - Fork 83
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
fix: accessibility issue [#4099] #4104
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to confirm, did you test to confirm this passes the a11y test? In my view, the problem was coming from the lack of ← in the aria-label, but you can correct me if I'm wrong.
The error says that "Visible text labels that do not match the accessible name can result in a confusing experience for screen reader users." |
…ix/accessibility-issue
@cristian-ungureanu tested and everything is working well 👍 |
🎉 This PR is included in version 3.7.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
aria-label
attribute.Will affect the visual aspect of the product
NO
Test instructions
Check before Pull Request is ready:
Closes #4099.