fix(visualMap): fix continuous visualMap mouseover event not register…

… except first countinous visualMap (#19097) * fix(visualMap): fix continuous visualMap mouseover event not register except first countinous visualMap * fix(visualMap): bind current instance to eventHandler to make unique function in init function * fix(visualMap): remove unnecessary declaration and assignment --------- Co-authored-by: plainheart <[email protected]>
apache · Sep 11, 2023 · a06dd62 · a06dd62
1 parent 428e2ad
commit a06dd62
Show file tree

Hide file tree

Showing 4 changed files with 331 additions and 15 deletions.
diff --git a/src/component/visualMap/ContinuousView.ts b/src/component/visualMap/ContinuousView.ts
@@ -108,17 +108,19 @@ class ContinuousView extends VisualMapView {
 
     private _firstShowIndicator: boolean;
 
-    private _api: ExtensionAPI;
+    init(ecModel: GlobalModel, api: ExtensionAPI) {
+        super.init(ecModel, api);
 
+        this._hoverLinkFromSeriesMouseOver = zrUtil.bind(this._hoverLinkFromSeriesMouseOver, this);
+        this._hideIndicator = zrUtil.bind(this._hideIndicator, this);
+    }
 
     doRender(
         visualMapModel: ContinuousModel,
         ecModel: GlobalModel,
         api: ExtensionAPI,
         payload: {type: string, from: string}
     ) {
-        this._api = api;
-
         if (!payload || payload.type !== 'selectDataRange' || payload.from !== this.uid) {
             this._buildView();
         }
@@ -711,7 +713,7 @@ class ContinuousView extends VisualMapView {
             for (let i = 0; i < handleLabels.length; i++) {
                 // Fade out handle labels.
                 // NOTE: Must use api enter/leave on emphasis/blur/select state. Or the global states manager will change it.
-                this._api.enterBlur(handleLabels[i]);
+                this.api.enterBlur(handleLabels[i]);
             }
         }
     }
@@ -856,7 +858,7 @@ class ContinuousView extends VisualMapView {
             for (let i = 0; i < handleLabels.length; i++) {
                 // Fade out handle labels.
                 // NOTE: Must use api enter/leave on emphasis/blur/select state. Or the global states manager will change it.
-                this._api.leaveBlur(handleLabels[i]);
+                this.api.leaveBlur(handleLabels[i]);
             }
         }
     }
@@ -874,6 +876,7 @@ class ContinuousView extends VisualMapView {
         this._hideIndicator();
 
         const zr = this.api.getZr();
+
         zr.off('mouseover', this._hoverLinkFromSeriesMouseOver);
         zr.off('mouseout', this._hideIndicator);
     }
@@ -892,7 +895,7 @@ class ContinuousView extends VisualMapView {
             : graphic.transformDirection(vertex, transform, inverse);
     }
 
- // TODO: TYPE more specified payload types.
+    // TODO: TYPE more specified payload types.
     private _dispatchHighDown(type: 'highlight' | 'downplay', batch: Payload['batch']) {
         batch && batch.length && this.api.dispatchAction({
             type: type,
@@ -908,14 +911,6 @@ class ContinuousView extends VisualMapView {
         this._clearHoverLinkToSeries();
     }
 
-    /**
-     * @override
-     */
-    remove() {
-        this._clearHoverLinkFromSeries();
-        this._clearHoverLinkToSeries();
-    }
-
 }
 
 function createPolygon(

diff --git a/test/runTest/actions/__meta__.json b/test/runTest/actions/__meta__.json
diff --git a/test/runTest/actions/visualMap-multi-continuous.json b/test/runTest/actions/visualMap-multi-continuous.json