[FIX] spreadsheet: fix zoomable chart cursor

Commit Description

These changes aims to fix some bad behaviors with the cursor for
the zoomable chart, caused by the onOdooChartItemHover method and
the odoo menu link for the Odoo chart.
This fix removes the onHover and onClick options for the master
chart in zoomable chart, and add an early return for the menu link
plugin in case of no menu or no dashboard mode.

Related Task

closes #8013

Task: 5012198
Signed-off-by: Pierre Rousseau (pro) <pro@odoo.com>

Author: anhe-odoo

src/components/figures/chart/chartJs/zoomable_chart/zoomable_chartjs.ts

@@ -455,6 +455,7 @@ export class ZoomableChartJsComponent extends ChartJsComponent {
       }
       this.updateAxisLimits(min, max);
       this.mode = undefined;
+      this.updateMasterChartCursor(ev);
     };
     this.removeEventListeners = () => {
       window.removeEventListener("pointermove", onMasterChartDrag, true);
@@ -465,7 +466,7 @@ export class ZoomableChartJsComponent extends ChartJsComponent {
     window.addEventListener("pointerup", onMasterChartPointerUp, true);
   }
 
-  onMasterChartPointerMove(ev: PointerEvent) {
+  updateMasterChartCursor(ev: PointerEvent) {
     const { offsetX: x, offsetY: y, target } = ev;
     if (!target || !this.isMasterChartAllowed) {
       return;

src/components/figures/chart/chartJs/zoomable_chart/zoomable_chartjs.xml

@@ -13,7 +13,7 @@
           t-ref="masterChartCanvas"
           t-on-dblclick="onMasterChartDoubleClick"
           t-on-pointerdown="onMasterChartPointerDown"
-          t-on-pointermove="onMasterChartPointerMove"
+          t-on-pointermove="updateMasterChartCursor"
           t-on-mouseleave="onMasterChartMouseLeave"
         />
       </div>