fix: tooltip.hide should pass the clientX and clientY

antvis · Dec 11, 2023 · ec656fc · ec656fc
1 parent fbd1a25
commit ec656fc
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/__tests__/integration/components/tooltip/tooltip-10.ts b/__tests__/integration/components/tooltip/tooltip-10.ts
@@ -49,7 +49,7 @@ export const Tooltip10 = () => {
     tooltip.show(e.offsetX, e.offsetY);
   });
   group.addEventListener('mouseleave', (e: any) => {
-    tooltip.hide(e.offsetX, e.offsetY);
+    tooltip.hide(e.clientX, e.clientY);
   });
 
   return group;

diff --git a/__tests__/integration/components/tooltip/tooltip-2.ts b/__tests__/integration/components/tooltip/tooltip-2.ts
@@ -64,7 +64,7 @@ export const Tooltip2 = () => {
     tooltip.show();
   });
   group.addEventListener('mouseleave', (e: any) => {
-    tooltip.hide(e.offsetX, e.offsetY);
+    tooltip.hide(e.clientX, e.clientY);
   });
   return group;
 };

diff --git a/src/ui/tooltip/index.ts b/src/ui/tooltip/index.ts
@@ -256,14 +256,16 @@ export class Tooltip extends Component<TooltipStyleProps> {
     return this.getRelativeOffsetFromCursor(correctedPositionString as TooltipPosition);
   }
 
-  private isCursorEntered(cursorX: number, cursorY: number) {
+  private isCursorEntered(clientX: number, clientY: number) {
     // 是可捕获的，并且点在 tooltip dom 上
     if (this.element) {
       const { x, y, width, height } = this.element.getBoundingClientRect();
-      const { container } = this.attributes;
-      const { x: cx, y: cy } = container;
+      // const { container } = this.attributes;
+      // const { x: cx, y: cy } = container;
 
-      return new BBox(x - cx, y - cy, width, height).isPointIn(cursorX, cursorY);
+      // console.log(1113, [clientX, clientY], [x, y, width, height], [cx, cy], new BBox(x - cx, y - cy, width, height).isPointIn(cursorX, cursorY));
+
+      return new BBox(x, y, width, height).isPointIn(clientX, clientY);
     }
     return false;
   }