Fixed a JavaScript error when clicking anywhere but on a toggle

edenspiekermann · Mar 14, 2016 · 1be89d0 · 1be89d0
1 parent 63884d6
commit 1be89d0
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/a11y-toggle.js b/a11y-toggle.js
@@ -58,10 +58,11 @@
     var toggle = event.target.hasAttribute(namespace)
       ? event.target
       : closest(event.target, '[' + namespace + ']');
-    var target = targetsMap[toggle.getAttribute(namespace)];
-    var isExpanded = toggle.getAttribute('aria-expanded') === 'true';
+    var target = toggle && targetsMap[toggle.getAttribute(namespace)];
 
     if (target) {
+      var isExpanded = toggle.getAttribute('aria-expanded') === 'true';
+
       toggle.setAttribute('aria-expanded', !isExpanded);
       target.setAttribute('aria-hidden', isExpanded);
     }

diff --git a/a11y-toggle.min.js b/a11y-toggle.min.js
diff --git a/example/main.js b/example/main.js
@@ -58,10 +58,11 @@
     var toggle = event.target.hasAttribute(namespace)
       ? event.target
       : closest(event.target, '[' + namespace + ']');
-    var target = targetsMap[toggle.getAttribute(namespace)];
-    var isExpanded = toggle.getAttribute('aria-expanded') === 'true';
+    var target = toggle && targetsMap[toggle.getAttribute(namespace)];
 
     if (target) {
+      var isExpanded = toggle.getAttribute('aria-expanded') === 'true';
+
       toggle.setAttribute('aria-expanded', !isExpanded);
       target.setAttribute('aria-hidden', isExpanded);
     }