odoo/o-spreadsheet#6314

Created by fw-bot

label

odoo:saas-18.2-saas-18.1-chart-legend-onclick-dashboard-v2-adrm-428544-fw

head

95323c50382e1125dcdae4654636559c7a82c745

merged

8 months ago by BI, Adrien Minne (adrm)

	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

[FW] 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

Forward-Port-Of: #6076