Added workaround for missing event.path

maciex · Aug 5, 2016 · 03e7ce5 · 03e7ce5
1 parent bc51773
commit 03e7ce5
Showing 1 changed file with 20 additions and 8 deletions.
diff --git a/js/content_scripts/context-menu-content-scripts.js b/js/content_scripts/context-menu-content-scripts.js
@@ -1,3 +1,13 @@
+function getEventPath(event) {
+    var path = [];
+    var node = event.target;
+    while(node != document.body) {
+        path.push(node);
+        node = node.parentNode;
+    }
+    return path;
+}
+
 function addContextMenuTo(selector) {
     selector = selector + ', ' + selector + ' *';
     var matches;
@@ -11,14 +21,16 @@ function addContextMenuTo(selector) {
     var isHovering = false;
     document.addEventListener('mouseover', function(event) {
         if (event.target && event.target[matches](selector)) {
-            if (event.path) {
-                for (var i = 0; i < event.path.length; i++) {
-                    var element = event.path[i];
-                    if (element[matches] && element[matches]('a')) {
-                        createContextMenu(element.href);
-                        isHovering = true;
-                        break;
-                    }
+            if (!event.path) {
+                console.log("Using workaround to get event.path");
+                event.path = getEventPath(event);
+            }
+            for (var i = 0; i < event.path.length; i++) {
+                var element = event.path[i];
+                if (element[matches] && element[matches]('a')) {
+                    createContextMenu(element.href);
+                    isHovering = true;
+                    break;
                 }
             }
         } else if (isHovering) {