-
Notifications
You must be signed in to change notification settings - Fork 9
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
Suggested accessibility improvements #1
Comments
Hi @Malvoz, ok ! I will work to improve this. Let me make a question: do you know why this plugin is not merge yet? 😢 |
The maintainers will merge it eventually, when they have the time. 🙂 |
Oh, thanks 👍 |
I wrote some accessibility-related content for Leaflet that you may find helpful: ✌🏼 |
Thanks you, It's very helpful |
Hi @clavijojuan, it's me again 😋
Please consider making the switches (e.g. the
<div class="basemapImg">
) keyboard focusable and understandable by screen reader users:tabindex="0"
can be used to make the icons focusablerole="button"
lets users with screen readers know that they can be pressedaria-pressed="true|false"
lets screen reader users know if the icon is selected or notThe text was updated successfully, but these errors were encountered: