diff options
| author | qsm-odoo <qsm@odoo.com> | 2020-08-06 16:13:56 +0000 |
|---|---|---|
| committer | qsm-odoo <qsm@odoo.com> | 2020-08-07 08:52:19 +0000 |
| commit | 201e10b338c8e50d7dd7d226676797eff6c26199 (patch) | |
| tree | 8547518e8b026ef204618d46f858d36f9a187781 /addons/web_editor/static/src/js/common/utils.js | |
| parent | 315ba1cfeb737c378ee6828cc6f612f1ab91fa71 (diff) | |
[FIX] web_editor: remove useless snippet preview notification
Before this commit, the 'snippet_option_update' event was triggered
whether or not it was a preview update. In the preview update case,
nothing was done, ever. This commit removes the preview notification.
If it ever becomes useful, it can be restored at that moment.
This commit actually solves a bug introduced by [1]: that snippet update
notification is now triggered after a setTimeout delay to allow events
to bubble before the UI update it will trigger (in non preview mode).
The side effect this had was that, between each preview, the execution
chain was deferred of a setTimeout delay between each step (even though
previews do not update the UI). This was enough to allow for a paint
refresh.
So before this commit, if option C was selected, then the user hovered
option A then option B, the option C was visually re-enabled between A
and B. This won't be the case anymore after this commit.
Long story short: restore no-flickering when previewing options.
[1]: https://github.com/odoo/odoo/commit/2616c8950fd615561e720660bcc39e018f4061e4
Part of https://github.com/odoo/odoo/pull/55588
closes odoo/odoo#55588
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
Diffstat (limited to 'addons/web_editor/static/src/js/common/utils.js')
0 files changed, 0 insertions, 0 deletions
