Merge pull request #75 from wanadev/fix-absolute-widget-position

take scroll into account in absolute position calculation
wanadev · Sep 14, 2016 · 3077bc6 · 3077bc6
2 parents 84257b2 + 8482b8a
commit 3077bc6
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/src/helpers.js b/src/helpers.js
@@ -106,12 +106,16 @@ Helpers.getAbsolutePosition = function (element) {
     if (!(element instanceof Element)) {
         return {x: 0, y: 0};
     }
+
     var css;
+    var origElement = element;
+
     try {
         css = getComputedStyle(element);
     } catch (e) {
         return {x: 0, y: 0};
     }
+
     if (!css) {
         return {x: 0, y: 0};
     }
@@ -131,6 +135,14 @@ Helpers.getAbsolutePosition = function (element) {
         element = element.offsetParent;
     }
 
+    element = origElement;
+
+    while (element.parentNode && !(element instanceof HTMLBodyElement)) {
+        x -= element.scrollLeft || 0;
+        y -= element.scrollTop || 0;
+        element = element.parentNode;
+    }
+
     return {x: x, y: y};
 };