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

[FIX] format: dispatch only once

Open LucasLefevre opened this issue 1 year ago • 1 comments

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

LucasLefevre avatar Aug 02 '24 11:08 LucasLefevre

Pull request status dashboard

robodoo avatar Aug 02 '24 11:08 robodoo

@robodoo r+

rrahir avatar Aug 06 '24 06:08 rrahir