fix: ensure tooltip is in view

btd · Mar 2, 2024 · 14bc982 · 14bc982
1 parent dec8398
commit 14bc982
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 9 deletions.
diff --git a/src/flamegraph/tooltip.tsx b/src/flamegraph/tooltip.tsx
@@ -130,12 +130,12 @@ export const Tooltip: FunctionalComponent<TooltipProps> = ({
 
     if (pos.left + boundingRect.width > window.innerWidth) {
       // Shifting horizontally
-      pos.left = window.innerWidth - boundingRect.width;
+      pos.left = Math.max(0,window.innerWidth - boundingRect.width);
     }
 
     if (pos.top + boundingRect.height > window.innerHeight) {
       // Flipping vertically
-      pos.top = mouseCoords.y - Tooltip_marginY - boundingRect.height;
+      pos.top = Math.max(0,mouseCoords.y - Tooltip_marginY - boundingRect.height);
     }
 
     setStyle(pos);

diff --git a/src/network/tooltip.tsx b/src/network/tooltip.tsx
@@ -50,12 +50,12 @@ export const Tooltip: FunctionalComponent<TooltipProps> = ({ node, visible }) =>
 
     if (pos.left + boundingRect.width > window.innerWidth) {
       // Shifting horizontally
-      pos.left = window.innerWidth - boundingRect.width;
+      pos.left = Math.max(window.innerWidth - boundingRect.width);
     }
 
     if (pos.top + boundingRect.height > window.innerHeight) {
       // Flipping vertically
-      pos.top = mouseCoords.y - Tooltip_marginY - boundingRect.height;
+      pos.top = Math.max(mouseCoords.y - Tooltip_marginY - boundingRect.height);
     }
 
     setStyle(pos);

diff --git a/src/style/_base.scss b/src/style/_base.scss
@@ -60,8 +60,6 @@ main {
 
   padding: 5px;
 
-  white-space: nowrap;
-
   font-size: 0.875rem;
 
   background-color: var(--background-color);
@@ -139,4 +137,4 @@ main {
 
 .node {
   cursor: pointer;
-}
+}
diff --git a/src/treemap/tooltip.tsx b/src/treemap/tooltip.tsx
@@ -130,12 +130,12 @@ export const Tooltip: FunctionalComponent<TooltipProps> = ({
 
     if (pos.left + boundingRect.width > window.innerWidth) {
       // Shifting horizontally
-      pos.left = window.innerWidth - boundingRect.width;
+      pos.left = Math.max(window.innerWidth - boundingRect.width);
     }
 
     if (pos.top + boundingRect.height > window.innerHeight) {
       // Flipping vertically
-      pos.top = mouseCoords.y - Tooltip_marginY - boundingRect.height;
+      pos.top = Math.max(mouseCoords.y - Tooltip_marginY - boundingRect.height);
     }
 
     setStyle(pos);