odoo/o-spreadsheet#7498

Created by BI, Rémi Rahir (rar)

label

odoo:master-avoid-input-on-selection-rar

head

83a72b89f841e03d5dd1dce8b9cece00b5bce033

merged

2 weeks ago by BI, Pierre Rousseau (pro)

	odoo/o-spreadsheet
master	#7498

[FIX] Selection: Ignore keyboard events when selecting range in the grid

Currently, users can modify the input of a composer/selection input while selecting a range in the grid. Those inputs can have side-effect that are difficult to account for as the selection is managed in a hook which is not aware of the related feature that it handles (selection, cell reference, pivot reference, selection input). A recent work to fix that: #7400

Overall, there is no added value to be able to mix writing text while selecting ranges inside the grid so we simply disable that possibility altogether.

task-5347297

Description:

description of this task, what is implemented and why it is implemented that way.

Task: TASK_ID

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