Actually fix issue #4 (touch/click issue) by fixing it in the source …

…files.
kylepaulsen · May 18, 2015 · e3f4c75 · josdejong · May 19, 2015 · e3f4c75
1 parent 5acbf5f
commit e3f4c75
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 16 deletions.
diff --git a/nanomodal.js b/nanomodal.js
@@ -43,16 +43,18 @@ function El(tag, classNames) {
     };
 
     var addClickListener = function(handler) {
-        var flag = false;
-        var wrappedHandler = function(evt){
-            if (!flag){
-                flag = true;
-                setTimeout(function(){ flag = false; }, 100);
-                handler(evt);
+        var throttle = false;
+        var throttleHandler = function(e) {
+            if (!throttle) {
+                throttle = true;
+                setTimeout(function() {
+                    throttle = false;
+                }, 100);
+                handler(e);
             }
-        }
-        addListener("touchstart", wrappedHandler);
-        addListener("mousedown", wrappedHandler);
+        };
+        addListener("touchstart", throttleHandler);
+        addListener("mousedown", throttleHandler);
     };
 
     var show = function(arg) {
@@ -376,7 +378,7 @@ var ModalEvent = require("./ModalEvent");
 
 var nanoModalAPI = (function() {
 
-
+    
 
     var El = require("./El");
     var Modal = require("./Modal");

diff --git a/nanomodal.min.js b/nanomodal.min.js
diff --git a/src/El.js b/src/El.js
@@ -41,11 +41,18 @@ function El(tag, classNames) {
     };
 
     var addClickListener = function(handler) {
-        if ("ontouchend" in document.documentElement) {
-            addListener("touchstart", handler);
-        } else {
-            addListener("click", handler);
-        }
+        var throttle = false;
+        var throttleHandler = function(e) {
+            if (!throttle) {
+                throttle = true;
+                setTimeout(function() {
+                    throttle = false;
+                }, 100);
+                handler(e);
+            }
+        };
+        addListener("touchstart", throttleHandler);
+        addListener("mousedown", throttleHandler);
     };
 
     var show = function(arg) {