fix(legacy-plugin-chart-sankey): ensure tooltip position is within ch…

…art (apache#1092)
zhaoyongjie · Nov 24, 2021 · e301e9d · e301e9d
1 parent 1e48622
commit e301e9d
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/...ontend/temporary_superset_ui/superset-ui/plugins/legacy-plugin-chart-sankey/src/Sankey.js b/...ontend/temporary_superset_ui/superset-ui/plugins/legacy-plugin-chart-sankey/src/Sankey.js
@@ -115,9 +115,11 @@ function Sankey(element, props) {
     tooltip
       .html(() => getTooltipHtml(d))
       .transition()
-      .duration(200)
-      .style('left', `${d3.event.offsetX + 10}px`)
-      .style('top', `${d3.event.offsetY + 10}px`)
+      .duration(200);
+    const { height: tooltipHeight, width: tooltipWidth } = tooltip.node().getBoundingClientRect();
+    tooltip
+      .style('left', `${Math.min(d3.event.offsetX + 10, width - tooltipWidth)}px`)
+      .style('top', `${Math.min(d3.event.offsetY + 10, height - tooltipHeight)}px`)
       .style('position', 'absolute')
       .style('opacity', 0.95);
   }