Merge pull request #2 from Malvoz/align-with-leaflet-core-controls

[accessibility] Align with Leaflet controls

Author: runette

CHANGELOG.md

@@ -1,4 +1,9 @@
-##1.0.4
+## 1.0.5
+
+* Add `role="button"` to the control, and lowercase "fullscreen" in the `title`
+attribute to align with Leaflet's core controls.
+
+## 1.0.4
 
 * reinstated backward compatibility with `fullscreenOptions`
 

README.md

@@ -49,8 +49,8 @@ To change fullscreen control text:
 ``` js
 map.addControl(new L.Control.Fullscreen({
     title: {
-        'false': 'View Fullscreen',
-        'true': 'Exit Fullscreen'
+        'false': 'View fullscreen',
+        'true': 'Exit fullscreen'
     }
 }));
 ```

dist/Leaflet.fullscreen.js

@@ -17,8 +17,8 @@
         options: {
             position: 'topleft',
             title: {
-                'false': 'View Fullscreen',
-                'true': 'Exit Fullscreen'
+                'false': 'View fullscreen',
+                'true': 'Exit fullscreen'
             }
         },
 
@@ -27,6 +27,7 @@
 
             this.link = L.DomUtil.create('a', 'leaflet-control-fullscreen-button leaflet-bar-part', container);
             this.link.href = '#';
+            this.link.setAttribute('role', 'button');
 
             this._map = map;
             this._map.on('fullscreenchange', this._toggleTitle, this);

dist/Leaflet.fullscreen.min.js

@@ -1,6 +1,6 @@
 {
   "name": "@runette/leaflet-fullscreen",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "A fullscreen control for Leaflet",
   "main": "dist/Leaflet.fullscreen.js",
   "directories": {