-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FIX] BottomBarSheet: Rename a sheet with style content on Firefox #3894
Conversation
The issue addressed in PR #3351 unfortunately used a `contentEditable` value that is not compatible with Firefox [1] This revision implements the first strategy explored by the aforementioned PR by removing the full format from the contenteditable span when we stop the edition. Testing required to define some properties as JSdom does not properly support the implementation of innerText (see [2]) [1]: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/contenteditable#browser_compatibility [2]: jsdom/jsdom#1245 Task: 3754944
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -12,6 +12,15 @@ export let OWL_TEMPLATES: Document; | |||
beforeAll(() => { | |||
OWL_TEMPLATES = getParsedOwlTemplateBundle(); | |||
setDefaultSheetViewSize(1000); | |||
Object.defineProperty(Element.prototype, "innerText", { | |||
get: function myProperty() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
get: function myProperty() { | |
get: function () { |
This whole syntax looks strange, but the function name doesn't seem mandatory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
robodoo r+
The issue addressed in PR #3351 unfortunately used a `contentEditable` value that is not compatible with Firefox [1] This revision implements the first strategy explored by the aforementioned PR by removing the full format from the contenteditable span when we stop the edition. Testing required to define some properties as JSdom does not properly support the implementation of innerText (see [2]) [1]: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/contenteditable#browser_compatibility [2]: jsdom/jsdom#1245 closes #3894 Task: 3754944 Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
The issue addressed in PR #3351 unfortunately used a
contentEditable
value that is not compatible with Firefox 1This revision implements the first strategy explored by the aforementioned PR by removing the full format from the contenteditable span when we stop the edition.
Testing required to define some properties as JSdom does not properly support the implementation of innerText (see 2)
Task: 3754944
Description:
description of this task, what is implemented and why it is implemented that way.
Task: : TASK_ID
review checklist