search

brunob / leaflet.fullscreen

Leaflet.Control.FullScreen for Leaflet
https://brunob.github.io/leaflet.fullscreen/
MIT License
376 stars 107 forks source link

Button Accessibility Issue #66

Closed sdlime closed 5 years ago

sdlime commented 5 years ago

Hi: The control throws an accessibility error when using the Wave Accessibility Tool (https://webaim.org). The issue is there's no text associated with the control button link. The fix is easy - add an aria-label attribute to the link so it becomes:

<a class="leaflet-control-zoom-fullscreen fullscreen-icon" href="#" title="Full Screen" aria-label="Full Screen" style="outline: none;"></a>

This would match the pattern leaflet uses with it's built in zoom in/out control.

--Steve

brunob commented 5 years ago

Fixed in 1.5.1, thx for the hint :)