-
Notifications
You must be signed in to change notification settings - Fork 123
Open
Description
aria-hidden
is used in the disabled state:
Leaflet.EasyButton/src/easy-button.js
Line 289 in cd53db5
this.button.setAttribute('aria-hidden', 'false'); |
Leaflet.EasyButton/src/easy-button.js
Line 296 in cd53db5
this.button.setAttribute('aria-hidden', 'true'); |
which only hides the content from screen readers, but the button remains focusable, and because it is aria-hidden="true"
nothing is announced to the user when it is focused.
I suggest replacing aria-hidden
with aria-disabled
to convey to screen readers that the button is currently disabled.
See related Leaflet issue: Leaflet/Leaflet#7211 / PR: https://github.com/Leaflet/Leaflet/pull/7280/files.
Other potentially helpful resources:
Metadata
Metadata
Assignees
Labels
No labels