summaryrefslogtreecommitdiff
path: root/addons/web
diff options
context:
space:
mode:
authorNicolas Lempereur <nle@odoo.com>2021-04-21 09:17:56 +0000
committerNicolas Lempereur <nle@odoo.com>2021-04-21 09:40:03 +0000
commitc7a4cca4a082af7c6b33a70d4b19e700e4d487e7 (patch)
tree8c6f15062ca5f4d79ba93cb62361ca10ffc39bd4 /addons/web
parent13832912daafad7c472d7414eaacede9e302e787 (diff)
[FIX] web: find scrollingElement despite rounding
In mass mailing edition, in some configuration of: - being in a modal or not - size of window - zoom level we might get an error when getting the scrolling element. For example when editing a mass mailing, if I set my viewport to 595 pixels height and have a 90% zoom, the editor is broken. This is because in SnippetsMenu.starts `$().getScrollingElement()` doesn't find an element that has the height of body. In my use case body height is 329.111 pixels, and scroll height is 328 pixels. Testing different zoom level and viewport height, the maximum difference I was able to get was 1.25 so this commit increase to 1.5 the maxmimum difference. opw-2455727 opw-2469174 closes odoo/odoo#69592 Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
Diffstat (limited to 'addons/web')
-rw-r--r--addons/web/static/src/js/libs/jquery.js2
1 files changed, 1 insertions, 1 deletions
diff --git a/addons/web/static/src/js/libs/jquery.js b/addons/web/static/src/js/libs/jquery.js
index 15917cede38..749bcd80133 100644
--- a/addons/web/static/src/js/libs/jquery.js
+++ b/addons/web/static/src/js/libs/jquery.js
@@ -171,7 +171,7 @@ $.fn.extend({
// Search for a body child which is at least as tall as the body
// and which has the ability to scroll if enough content in it. If
// found, suppose this is the top scrolling element.
- if (bodyHeight - el.scrollHeight > 1) {
+ if (bodyHeight - el.scrollHeight > 1.5) {
continue;
}
const $el = $(el);