From 7a8c0b0091be3f3d1177e84536f3dbc4b9520ef9 Mon Sep 17 00:00:00 2001 From: Rodolpho Lima Date: Fri, 3 Nov 2023 17:29:02 +0100 Subject: [PATCH] [FIX] web_editor: remove observerUnactive before adding tooltip Since [1], the tooltip is not longer the jQuery-UI or Bootstrap one, but the popover service with the Tooltip component, which does not make changes to the editable DOM when being added or removed. This means that switching the observer off before opening the tooltip is no longer necessary. [1]: https://github.com/odoo/odoo/commit/08cd27cc8ad78d2ee9ffe80ddb9e1cc50a557ee2 closes odoo/odoo#141882 Signed-off-by: Nicolas Bayet (nby) --- addons/web_editor/static/src/js/wysiwyg/wysiwyg.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/addons/web_editor/static/src/js/wysiwyg/wysiwyg.js b/addons/web_editor/static/src/js/wysiwyg/wysiwyg.js index 5b5653c885c..add61a34f1e 100644 --- a/addons/web_editor/static/src/js/wysiwyg/wysiwyg.js +++ b/addons/web_editor/static/src/js/wysiwyg/wysiwyg.js @@ -2178,9 +2178,7 @@ export class Wysiwyg extends Component { } // Tooltips need to be cleared before leaving the editor. this.saving_mutex.exec(() => { - this.odooEditor.observerUnactive(); const removeTooltip = this.popover.add(e.target, Tooltip, { tooltip: _t('Double-click to edit') }); - this.odooEditor.observerActive(); this.tooltipTimeouts.push(setTimeout(() => removeTooltip(), 800)); }); }, 400));