diff options
| author | Romeo Fragomeli <rfr@odoo.com> | 2019-09-30 11:15:31 +0000 |
|---|---|---|
| committer | Romeo Fragomeli <rfr@odoo.com> | 2019-09-30 12:27:57 +0000 |
| commit | 53bb1387b58937f2b6b84768f59e6982f7847517 (patch) | |
| tree | adf1480d700e81627d32e1c32b3fee5d81065eea /addons/web/static/src/js/views/basic | |
| parent | fdc3555c8d3967f03fa600134d5c84932cea82ef (diff) | |
[FIX] web: control panel: error by clicking on padding on mobile
o_control_panel have this layout:
+-------------------------------------+
| PADDING |
| +----------------------------------+
| | breadcrumb o_cp_searchview |
| | o_cp_left o_cp_right |
| +----------------------------------+
+-------------------------------------+
As you can see, there is a padding on the edge of the widget.
In Odoo 12.0, before the control panel refactoring (odoo/odoo@40dd121938552d3f88289d2e5fcd1d57a0dd01bb)
the search view was a widget by itself (addons/web/static/src/js/views/search/search_view_mobile.js).
Note that in this case, once the following condition:
"ev.target === this.el" was reach the normal behavior
was executed.
But now the search view is embedded in the control panel.
There is a side effect: clicking on the padding of the control panel
triggers the "_onOpenMobileSearchView" and the condition is always true.
Therefore we try to open the search more even if it doesn't exist.
But anyway, the search view (full) shouldn't be opened when we click on
the control panel.
After this commit, the listener is scoped to o_toggle_searchview_full,
to avoid this side effect.
Steps to reproduce:
* Open any app with a control panel (e.g. accounting)
* Select a record (e.g. an invoice)
* Click on the padding of the control panel (BUG)
closes odoo/odoo#37587
Signed-off-by: Adrien Dieudonné (adr) <adr@odoo.com>
Diffstat (limited to 'addons/web/static/src/js/views/basic')
0 files changed, 0 insertions, 0 deletions
