o-spreadsheet icon indicating copy to clipboard operation
o-spreadsheet copied to clipboard

[FIX] clipboard: paste multiline text from OS clipboard

Open hokolomopo opened this issue 10 months ago • 1 comments

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

hokolomopo avatar Feb 12 '25 13:02 hokolomopo

Pull request status dashboard

robodoo avatar Feb 12 '25 13:02 robodoo