odoo/o-spreadsheet#7184

Created by BI, Rémi Rahir (rar)

label

odoo:18.0-close-side-panel-on-dup-chart-rar

head

be6f8b9c036cd16ce351e92599e5982edc739af7

merged

4 days ago by BI, Vincent Schippefilt (vsc)

	odoo/o-spreadsheet
18.0	#7184
saas-18.2	#7476
saas-18.3	#7478
saas-18.4	#7480
19.0	#7481
master	#7482

[FIX] SidePanel: reset initial props on model change

Currently, we store the initial props with which the sidepanel was open. This specifically occurs for charts sidepanel where we want the stat to update when we select another chart BUT we also want it to stay open when clicking the grid (hence - no active chart left).

However the moment we click on another chart than the first one, then it is supposed to become the new "default" chart.

How to reproduce:
- open the sidepanel of the first chart
- select the second chart
- delete the second chart

=> the sidepanel is still open and falls back on the first chart instead of closing.

task-5059484

Description:

description of this task, what is implemented and why it is implemented that way.

Task: TASK_ID

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