Correct turn of the modal window from getModal
authorBen Schwarz <ben@germanforblack.com>
Sat, 12 Jan 2013 23:59:18 +0000 (10:59 +1100)
committerBen Schwarz <ben@germanforblack.com>
Sat, 12 Jan 2013 23:59:18 +0000 (10:59 +1100)
* Looks like I hastily changed it during review with @fat. Oops! :-)

lib/js/modals.js

index 51f0770..c796d48 100644 (file)
@@ -8,7 +8,6 @@
   var findModals = function (target) {
     var i;
     var modals = document.querySelectorAll('a');
-
     for (; target && target !== document; target = target.parentNode) {
       for (i = modals.length; i--;) { if (modals[i] === target) return target; }
     }
@@ -16,7 +15,7 @@
 
   var getModal = function (event) {
     var modalToggle = findModals(event.target);
-    if (!modalToggle || !modalToggle.hash) return document.querySelector(modalToggle.hash);
+    if (modalToggle && modalToggle.hash) return document.querySelector(modalToggle.hash);
   };
 
   window.addEventListener('touchend', function (event) {