odoo/o-spreadsheet#7183
Created by Ronakkumar Mukeshbhai Bharadiya (rmbh)
- label
- odoo:master-ref-cf-dv-side-panel-rmbh
- head
- 1cdf54f62effab09d37af1b9ccc1531bd71a9db7
odoo/o-spreadsheet | |
---|---|
master | #7183 missing r+ |
[REF] side_panel: split CF/DV panel into main and editor panel
Description:
Current behavior before PR:
- CF and DV side panels combined the list and editor into a single panel.
- The editor could not be opened directly.
- Logic for the main panel and editor was mixed, reducing modularity.
Desired behavior after PR is merged:
- Split CF and DV into separate main and editor panels.
- Panels can be opened or replaced directly with openSidePanel / replaceSidePanel.
- Use a store to manage the CF editor state.
- Refactor related test cases.
Task: 4982596
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