odoo/o-spreadsheet#4775
Created by fw-bot
Merged
at a32470f450beaeb10000541737211859a5f4faa1
Statuses:
- label
- odoo:saas-17.2-17.0-set-decimal-single-dispatch-lul-6719-fw
- head
- cac050406d3cd884a9716f9f1db5359338a4ab87
- merged
- 10 months ago by BI, Rémi Rahir (rar)
odoo/o-spreadsheet | |
---|---|
17.0 | #4770 |
saas-17.4 | #4776 |
saas-17.1 | #4774 |
saas-17.2 | #4775 |
18.0 | |
saas-18.1 | |
saas-18.2 | |
saas-18.3 | |
master | #4781 |
[FW][FIX] format: dispatch only once
Description:
Steps to reproduce:
- select a huge zone with numbers
- click on the "Increase decimal places" tool in the top bar
=> there's one dipatched command for each position in the zone, which can huge.
We want:
- one dispatched command per format
- positions grouped to a single zone if there are contiguous (a single big zone instead of many zones, each being actually a single position)
Issue introduced by 1b667256cc386e4b8e8ff2e216e66bd9b9a03d62
Task: : 4091502
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: #4770