diff options
| author | qsm-odoo <qsm@odoo.com> | 2017-05-15 17:18:18 +0200 |
|---|---|---|
| committer | qsm-odoo <qsm@odoo.com> | 2017-05-16 11:17:13 +0200 |
| commit | 1545995b393a1d347c6c1988da03dd1c6f1ea037 (patch) | |
| tree | 5c83ce51f945265c1858b0c89ddabc3b944d076f /addons/web/static/src/xml/base.xml | |
| parent | ab8c81a96fa99854975d3d3922521b5b944b5828 (diff) | |
[FIX] web_tour: tips were overlapping the chat windows
Before this commit, the tour tips were overlapping the chat windows.
This was due to the fact they always had a z-index of 1051 wherever
they were. If a tip was placed in a location where none of its
ancestors created a new stacking context, the closed tip was then
positioned above everything else, including the chat windows.
A simple solution would have been the one proposed by this commit:
https://github.com/odoo/odoo/commit/856e78b35a47a8732e5bd4da1cce247df70e5414
However, as it explained, this would not be entirely correct. In the
website, elements are using z-indexes and would thus hide closed
tooltips when it is not wanted if those have no z-index at all.
The solution this commit is making is to force the creation of a
stacking context for tooltips' parent. The z-index will still exist
on all tips (closed and not closed) to force them above every other
elements, but now only elements of the same parent. This is indeed
more correct (but maybe still not entirely): website tooltip are
often placed in the body (as it is the element which scrolls) and
are thus still placed above elements like the navbar as their
z-index is left high. However, when placed in the appswitcher, they
will now be placed above appswitcher elements only (and thus not
above outside elements which are themself above the appswitcher
(e.g. the chat windows)).
opw-743330
Diffstat (limited to 'addons/web/static/src/xml/base.xml')
0 files changed, 0 insertions, 0 deletions
