Fixed #893, #911: the `clickToUse` option of Network, Graph2d, and Network was blocking click events in the web page.

HISTORY.md

@@ -3,6 +3,11 @@ http://visjs.org
 
 ## not-yet-released, version 4.1.1-SNAPSHOT
 
+### General
+
+- Fixed #893, #911: the `clickToUse` option of Network, Graph2d, and Network 
+  was blocking click events in the web page.  
+
 ### Timeline
 
 - Improved function `fit()` to take into account the actual width of items.

lib/shared/Activator.js

@@ -41,14 +41,15 @@ function Activator(container) {
     });
   });
 
-  // attach a tap event to the window, in order to deactivate when clicking outside the timeline
-  this.bodyHammer = Hammer(document && document.body, {prevent_default: false});
-  this.bodyHammer.on('tap', function (event) {
-    // deactivate when clicked outside the container
-    if (!_hasParent(event.target, container)) {
-      me.deactivate();
-    }
-  });
+  // attach a click event to the window, in order to deactivate when clicking outside the timeline
+  if (document && document.body) {
+    this.onClick = function (event) {
+      if (!_hasParent(event.target, container)) {
+        me.deactivate();
+      }
+    };
+    document.body.addEventListener('click', this.onClick);
+  }
 
   if (this.keycharm !== undefined) {
     this.keycharm.destroy();
@@ -74,9 +75,14 @@ Activator.prototype.destroy = function () {
   // remove dom
   this.dom.overlay.parentNode.removeChild(this.dom.overlay);
 
+  // remove global event listener
+  if (this.onClick) {
+    document.body.removeEventListener('click', this.onClick);
+  }
+
   // cleanup hammer instances
+  this.hammer.destroy();
   this.hammer = null;
-  this.bodyHammer = null;
   // FIXME: cleaning up hammer instances doesn't work (Timeline not removed from memory)
 };
 

package.json

@@ -34,7 +34,7 @@
     "keycharm": "^0.2.0",
     "uuid": "^2.0.1",
     "moment": "^2.10.2",
-    "propagating-hammerjs": "^1.4.1"
+    "propagating-hammerjs": "^1.4.2"
   },
   "devDependencies": {
     "babel": "^5.1.11",