odoo/o-spreadsheet#6076

Created by BI, Adrien Minne (adrm)

label

odoo:saas-18.1-chart-legend-onclick-dashboard-v2-adrm

head

0379a0b7e9924724a2ecb30251da2dc7a0a9ec7e

merged

8 months ago by BI, Rémi Rahir (rar)

	odoo/o-spreadsheet
saas-18.1	#6076
saas-18.2	#6314
saas-18.3	#6323
saas-18.4
19.0
saas-19.1
master	#6328

Saas 18.1 chart legend onclick dashboard v2 adrm

[IMP] chart: add mouseup event to listened events

To detect a middle mouse click on a chart, we have to use the
mouseup event, the click event is not triggered with middle
mouse button.

[FIX] charts: unregister ChartJS extension on unmount

The ChartJS extension were not unregistered when the leaving the
spreadsheet. Some of them only make sense in the context of the
spreadsheet, so keeping them when leaving could lead to crashes
when using ChartJS elsewhere.

Task: 4636147

review checklist

• [ ] feature is organized in plugin, or UI components
• [ ] support of duplicate sheet (deep copy)
• [ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
• [ ] in model/UI: ranges are strings (to show the user)
• [ ] undo-able commands (uses this.history.update)
• [ ] multiuser-able commands (has inverse commands and transformations where needed)
• [ ] new/updated/removed commands are documented
• [ ] exportable in excel
• [ ] translations (_t("qmsdf %s", abc))
• [ ] unit tested
• [ ] clean commented code
• [ ] track breaking changes
• [ ] doc is rebuild (npm run doc)
• [ ] status is correct in Odoo