odoo/o-spreadsheet#8208
Created by fw-bot
Merged
at 14a0b8049f1de6e61b02d994dcd7d3b252210ac0
Statuses:
- label
- odoo:saas-18.4-18.0-fix-wrapping-wrap-dhrp-512039-fw
- head
- 8b093a01e5629a756cf8f45945dc0336c216f4b8
- merged
- 1 week ago by BI, Rémi Rahir (rar)
| odoo/o-spreadsheet | |
|---|---|
| 18.0 | #8117 |
| saas-18.2 | #8205 |
| saas-18.3 | #8206 |
| saas-18.4 | #8208 |
| 19.0 | #8213 |
| saas-19.1 | #8214 |
| saas-19.2 | #8215 |
| master | #8216 |
[FIX] grid_renderer_store: keep wrapping width with explicit align
Description:
Since PR#8004 ([FIX] Dynamic pivot: header alignment), width-aware
formatting is disabled on explicitly aligned cells to preserve the pivot
header alignment fix.
The renderer was reusing that same width to split wrapped content, so
cells with wrapping: "wrap" and an explicit horizontal alignment were
still getting the correct row height, but their text was rendered as a
single clipped line.
This commit separates the formatting width from the wrapping width:
- keep formatWidth disabled for explicitly aligned cells
- always use the real cell width to split wrapped text
Task: 6032407
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: #8117