bugfix: issue 11801

Signed-off-by: Saber Sun <saber.sun@fmr.com>
chartjs · Aug 27, 2024 · 66a5204 · 66a5204
1 parent 0ca1b6e
commit 66a5204
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/plugins/plugin.tooltip.js b/src/plugins/plugin.tooltip.js
@@ -630,13 +630,13 @@ export class Tooltip extends Element {
     return tooltipItems;
   }
 
-  update(changed, replay) {
+  update(changed, replay, inChartArea = true) {
     const options = this.options.setContext(this.getContext());
     const active = this._active;
     let properties;
     let tooltipItems = [];
 
-    if (!active.length) {
+    if (!active.length || !inChartArea) {
       if (this.opacity !== 0) {
         properties = {
           opacity: 0
@@ -1147,7 +1147,7 @@ export class Tooltip extends Element {
     const positionChanged = this._positionChanged(active, e);
 
     // Remember Last Actives
-    const changed = replay || !_elementsEqual(active, lastActive) || positionChanged;
+    const changed = replay || !inChartArea || !_elementsEqual(active, lastActive) || positionChanged;
 
     // Only handle target event on tooltip change
     if (changed) {
@@ -1159,7 +1159,7 @@ export class Tooltip extends Element {
           y: e.y
         };
 
-        this.update(true, replay);
+        this.update(true, replay, inChartArea);
       }
     }
 

diff --git a/test/specs/plugin.tooltip.tests.js b/test/specs/plugin.tooltip.tests.js
@@ -965,7 +965,7 @@ describe('Plugin.Tooltip', function() {
 
     // First dispatch change event, should update tooltip
     await jasmine.triggerMouseEvent(chart, 'mousemove', firstPoint);
-    expect(tooltip.update).toHaveBeenCalledWith(true, undefined);
+    expect(tooltip.update).toHaveBeenCalledWith(true, undefined, true);
 
     // Reset calls
     tooltip.update.calls.reset();
@@ -1028,15 +1028,15 @@ describe('Plugin.Tooltip', function() {
 
     // First dispatch change event, should update tooltip
     await jasmine.triggerMouseEvent(chart, 'mousemove', firstPoint);
-    expect(tooltip.update).toHaveBeenCalledWith(true, undefined);
+    expect(tooltip.update).toHaveBeenCalledWith(true, undefined, true);
 
     // Reset calls
     tooltip.update.calls.reset();
 
     // Second dispatch change event (same event), should update tooltip
     // because position mode is 'nearest'
     await jasmine.triggerMouseEvent(chart, 'mousemove', secondPoint);
-    expect(tooltip.update).toHaveBeenCalledWith(true, undefined);
+    expect(tooltip.update).toHaveBeenCalledWith(true, undefined, true);
   });
 
   describe('positioners', function() {