diff options
| author | Romeo Fragomeli <rfr@odoo.com> | 2020-02-21 09:19:34 +0000 |
|---|---|---|
| committer | Pierre Paridans <app@odoo.com> | 2020-02-21 15:20:18 +0000 |
| commit | 7e4774222515da63da9d78556bb1cdf850088497 (patch) | |
| tree | 3841e7da3ce1f580d64d2bbe09eda8d5fc6d7fed /addons/web/static/src/js/views/basic/basic_controller.js | |
| parent | 8f70f203d0f801ce4f6b693fc95815af9eb5226c (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
