[FIX] web_editor: fix display of double overlays in edit mode
Steps to reproduce the bug: 1. Drag and drop the "image - text" snippet. 2. Click on the image => you have a double overlay which appears (one on the column with handles and one on the image without handles), this is good, that's what we want: the ability to control the column width but as the image has options, it should be framed with an overlay too to indicate what you are editing in the right panel. 3. Hover the column options => good, only the overlay of the column remains in preview mode (without handles / buttons). 4. Hover the image options => good, only the overlay of the image remains in preview mode (without handles / buttons). 5. Stop hovering the panel => the double overlay should be restored but that's not the case: only the column one remains, even though the image options are still displayed in the panel. When we introduced the possibility to display 2 overlays at the same time in this commit [1], we added a loop to search for the overlays to display since there can now be several. But in a later commit [2], we added a `break` in this loop, which no longer allows displaying more than one overlay when necessary. [1]: https://github.com/odoo/odoo/commit/9bbe5be3fb0d995374a369851df3641979d8e553 [2]: https://github.com/odoo/odoo/commit/806a8db35b5e0e6a461422f5bba7c97180c3ef29 task-2919164 closes odoo/odoo#108334 X-original-commit: ee86f5a94965ce691393f5300bdfb6dc09780b1d Signed-off-by: Arthur Detroux (ard) <ard@odoo.com>
This commit is contained in:
@@ -2454,7 +2454,6 @@ var SnippetsMenu = Widget.extend({
|
||||
if (editor.isSticky()) {
|
||||
editor.toggleOverlay(true, false);
|
||||
customize$Elements = await editor.toggleOptions(true);
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user