summaryrefslogtreecommitdiff
path: root/addons/web/static/src/js/views/basic
diff options
context:
space:
mode:
authorRomeo Fragomeli <rfr@odoo.com>2019-09-30 11:15:31 +0000
committerRomeo Fragomeli <rfr@odoo.com>2019-09-30 12:27:57 +0000
commit53bb1387b58937f2b6b84768f59e6982f7847517 (patch)
treeadf1480d700e81627d32e1c32b3fee5d81065eea /addons/web/static/src/js/views/basic
parentfdc3555c8d3967f03fa600134d5c84932cea82ef (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