Code review comment for lp:~openerp-dev/openerp-web/7.0-bug-1095283-bth

Revision history for this message
Fabien Meghazi (OpenERP) (fme) wrote :

Please, merge your jquery selectors when possible :

if ($(ev.target).parents('.oe_searchview').length === 0 && $(ev.target).parents('div#ui-datepicker-div').length === 0) { ...

could be

if (!$(ev.target).parents('.oe_searchview, div#ui-datepicker-div').length) { ...

review: Needs Fixing

« Back to merge proposal