From aa887ed02ce864cd1036f39f7bc6b984b60578d0 Mon Sep 17 00:00:00 2001 From: Aaron Bohy Date: Wed, 29 Jul 2015 09:11:53 +0200 Subject: [FIX] web,mail: widgets: replace $el.toggle() by do_toggle() Since rev. 16eb36ff, do_show() and do_hide() on widgets toggle the classname o_hidden on which the rule 'display: none' is defined, instead of calling jQuery's show() and hide(). The method do_toggle(), sharing the same logic, has been introduced in rev. 74c5f772. From now on, every widgets should call do_toggle() instead of $el.toggle() as it will have undesiderable effects (they should call do_show() and do_hide() instead of $el.show() and $el.hide() as well, this as been done in 16eb36ff). This rev. replaces all calls to $el.toggle() by do_toggle(). It also adds an optional argument to do_toggle() to force hidding or showing the widget. --- addons/web/static/src/css/base.css | 1 - addons/web/static/src/css/base.sass | 1 - 2 files changed, 2 deletions(-) (limited to 'addons/web/static/src/css') diff --git a/addons/web/static/src/css/base.css b/addons/web/static/src/css/base.css index fa2772f453f..55299c21687 100644 --- a/addons/web/static/src/css/base.css +++ b/addons/web/static/src/css/base.css @@ -1111,7 +1111,6 @@ .openerp .oe_searchview { cursor: text; position: relative; - display: none; height: auto; padding-top: 1px; padding-bottom: 1px; diff --git a/addons/web/static/src/css/base.sass b/addons/web/static/src/css/base.sass index f64c0985fea..0b2be908b4e 100644 --- a/addons/web/static/src/css/base.sass +++ b/addons/web/static/src/css/base.sass @@ -963,7 +963,6 @@ $sheet-padding: 16px .oe_searchview cursor: text position: relative - display: none height: auto padding-top: 1px padding-bottom: 1px -- cgit v1.2.3