summaryrefslogtreecommitdiff
path: root/addons/web/static/src/js/views/basic/basic_controller.js
diff options
context:
space:
mode:
authorRomeo Fragomeli <rfr@odoo.com>2020-02-21 09:19:34 +0000
committerPierre Paridans <app@odoo.com>2020-02-21 15:20:18 +0000
commit7e4774222515da63da9d78556bb1cdf850088497 (patch)
tree3841e7da3ce1f580d64d2bbe09eda8d5fc6d7fed /addons/web/static/src/js/views/basic/basic_controller.js
parent8f70f203d0f801ce4f6b693fc95815af9eb5226c (diff)
[FIX] mail: fix scroll in discuss on small screens
When using the Discuss app (not the systray's thread window) on a small device, it is not possible to scroll through the discussion (cf. tab "Chat" or "Channels"). Steps to reproduce: * Go in discuss (in mobile) * Go to tab chat or channel => BUG NB: you must have enough element to test the scroll This bug is due to the commit odoo/odoo@3ad77065605459140c70c2f5ffadceed92e417ca. Before it, the styles as interpreted by the browser looked like this: .o_mail_discuss_tab_pane { overflow-x:scroll; // explicitly set in discuss.scss overflow-y:auto; // implicit ; due to the overflow-x rule } But by removing the overflow-x rule, the styles ware interpreted as: .o_mail_discuss_tab_pane { overflow-x:visible; // implicit ; due to the absence of overflow-* rule overflow-y:visible; // implicit ; due to the absence of overflow-* rule } In addition, the parent element had an overflow: auto rule scoped only for desktop-like screens. This commit fixes this issue by unscoping the overflow: auto rule on the parent, allowing to scroll the messages' list in both desktop-like and mobile-like screens. Reference: https://developer.mozilla.org/en-US/docs/Web/CSS/overflow-x closes odoo/odoo#45964 X-original-commit: f4b22af7ed28d3515d77bfd76691aba1129581b1 Signed-off-by: Pierre Paridans <pparidans@users.noreply.github.com>
Diffstat (limited to 'addons/web/static/src/js/views/basic/basic_controller.js')
0 files changed, 0 insertions, 0 deletions