Fixing up position of tooltips for pie charts when page is scrolled

dc-js · Nov 19, 2015 · 819f75c · 819f75c
1 parent 1e6d6d0
commit 819f75c
Show file tree

Hide file tree

Showing 9 changed files with 17 additions and 17 deletions.
diff --git a/dc.js b/dc.js
diff --git a/dc.js.map b/dc.js.map
diff --git a/dc.min.js b/dc.min.js
diff --git a/dc.min.js.map b/dc.min.js.map
diff --git a/src/base-mixin.js b/src/base-mixin.js
@@ -1334,13 +1334,13 @@ dc.baseMixin = function (_chart) {
             var gBounding = _chart.g().node().getBoundingClientRect();
             var centroid = arc.centroid(d);
 
-            style.left = gBounding.left + (gBounding.width / 2) - (tooltipBounding.width / 2) + centroid[0];
-            style.top = gBounding.top + (gBounding.height / 2) - tooltipBounding.height - 10 + centroid[1];
+            style.left = gBounding.left + (gBounding.width / 2) - (tooltipBounding.width / 2) + centroid[0] + window.scrollX;
+            style.top = gBounding.top + (gBounding.height / 2) - tooltipBounding.height - 10 + centroid[1] + window.scrollY;
         // all other charts
         } else {
             var elBounding = element.getBoundingClientRect();
 
-            style.left = elBounding.left - (tooltipBounding.width / 2) + window.scrollX + (elBounding.width / 2);
+            style.left = elBounding.left + (elBounding.width / 2) - (tooltipBounding.width / 2) + window.scrollX;
             style.top = elBounding.top - tooltipBounding.height - 10 + window.scrollY;
         }
 

diff --git a/web/js/dc.js b/web/js/dc.js
diff --git a/web/js/dc.js.map b/web/js/dc.js.map
diff --git a/web/js/dc.min.js b/web/js/dc.min.js
diff --git a/web/js/dc.min.js.map b/web/js/dc.min.js.map