odoo/o-spreadsheet#4781
Created by fw-bot
Merged
at e9f817421ce9a3c643f76e917c19d1bb44d55347
Statuses:
- label
- odoo:master-17.0-set-decimal-single-dispatch-lul-2m9o-fw
- head
- 7983f1dddbcf351f999152479fd825298b606ae3
- 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