odoo/o-spreadsheet#7331

Created by BI, Rémi Rahir (rar)

Blocked

label
odoo:saas-18.4-fix-contenteditable-plaintext-only-behaviour-rar
head
97b4839d5c637a215251bc6fd584edd5a2ced284
odoo/o-spreadsheet
saas-18.4 #7331 missing statuses missing r+
19.0
master

[FIX] composer: fix get/setText of the content editable helper

The recent use of the property "plaintext-only" came with a change of behaviour in the composers, specifically in Firefox.

How to reproduce in FF:
- write a single character in a grid composer
- delete the character

-> a new line is inserted

Apparently, when setting the attribute contentEditable to plaintext-only, deleting the single character does not delete the span that encapsulates it if the latter has a class attribute. This couples with a recent refactoring of the content editable helper to handle new line characters and the empty composer is not detected as such

task-5082601

Description:

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

Task: 5082601

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