odoo/o-spreadsheet#5705
Created by BI, Adrien Minne (adrm)
- label
- odoo:18.0-multiline-os-clipboard-adrm
- head
- 38ebcd9bca7fd40955f7c3f9185ddaefece822ad
| odoo/o-spreadsheet | |
|---|---|
| 18.0 | #5705 missing r+ |
| saas-18.2 | |
| saas-18.3 | |
| saas-18.4 | |
| 19.0 | |
| saas-19.1 | |
| master |
[FIX] clipboard: paste multiline text from OS clipboard
Description
The implementation of the paste from OS clipboard a simple split on \n and \t to get the rows and columns in the clipboard. This obviously doesn't work if the cells copied from Excel contain newlines or tabs.
This commit fixes it by parsing the HTML content of the clipboard instead of the plain text. This makes it clear what the separation of row/cols are in the clipboard.
Task: 4570998
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