fix: height off tooltip should be element height

qlik-oss · Feb 18, 2020 · ccde196 · ccde196
1 parent 317b93a
commit ccde196
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/src/index.js b/src/index.js
@@ -185,6 +185,7 @@ export default function supernova(env) {
             selectionsAndTransform,
             selectionState,
             useTransitions: expandedState.useTransitions,
+            element,
           });
         }
       }, [expandedState, objectData, selectionState]);

diff --git a/src/tree/box.js b/src/tree/box.js
@@ -67,7 +67,7 @@ export default function box(
   selectionState,
   selectionsAndTransform,
   navigationMode,
-  height
+  element
 ) {
   const {
     cardWidth,
@@ -98,7 +98,7 @@ export default function box(
   function getTooltipStyle(d) {
     const halfCardWidth = cardWidth / 2;
     const halfTooltipWidth = tooltipWidth / 2;
-    return `bottom:${height -
+    return `bottom:${element.clientHeight -
       (y(d) * selectionsAndTransform.transform.zoom + selectionsAndTransform.transform.y - tooltipPadding)}px;left:${x(
       d
     ) *

diff --git a/src/tree/render.js b/src/tree/render.js
@@ -41,6 +41,7 @@ export const paintTree = ({
   selectionsAndTransform,
   selectionState,
   useTransitions,
+  element,
 }) => {
   const { svg, divBox, allNodes, positioning, width, height, tooltip } = objectData;
   const { navigationMode } = allNodes.data;
@@ -60,7 +61,7 @@ export const paintTree = ({
     selectionState,
     selectionsAndTransform,
     navigationMode,
-    height
+    element
   );
   // Create the lines (links) between the nodes
   const node = svg