odoo/o-spreadsheet#6328
Created by fw-bot
Merged
at 12dcb29ba1d6bc9f9016b11c6114eed047bb0a47
Statuses:
- label
- odoo:master-saas-18.1-chart-legend-onclick-dashboard-v2-adrm-434396-fw
- head
- faddbbb03df29e2b286e05502caab8a7ef76acce
- 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