diff options
| author | qsm-odoo <qsm@odoo.com> | 2020-09-24 13:56:55 +0000 |
|---|---|---|
| committer | qsm-odoo <qsm@odoo.com> | 2020-09-24 14:44:23 +0000 |
| commit | 5ffb7bb2f461de2b3cf95ef0df6a24b668bf9bd3 (patch) | |
| tree | 585a828c4d22680132a776ab40ba5296e9b27c47 /addons/website/static/src/xml/theme_preview.xml | |
| parent | 9f9419436a8e18a1802fabba1236b2ca5b5cc25a (diff) | |
[FIX] web: remove 'overlay' overflow on modals
The o-scrollbar-overlay mixin was introduced for a fix on the enterprise
home menu (see related commit in enterprise that comes with this PR).
It was used on modal elements at the same time for no apparent reason:
the modal in the backend do not even scroll as this is their modal-body
that does.
This now introduced a bug in the frontend: some code which allows to
scroll to an element in a modal was not working on chrome because the
related code was checking for the element with 'overflow: auto'... which
it could not find since that element uses 'overflow: overlay'. As
overlay is a deprecated value, it seemed better to remove its use rather
than adapt the frontend code expecting the 'auto' value. This also solve
a design inconsistency between Chrome and Firefox (where the overlay
value is simply ignored).
closes odoo/odoo#58444
Related: odoo/enterprise#13524
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
Diffstat (limited to 'addons/website/static/src/xml/theme_preview.xml')
0 files changed, 0 insertions, 0 deletions
