Merge lp:~openerp-dev/openerp-web/mck-opw-582226-cbi into lp:~openerp-dev/openerp-web/6.1-web-custo-tfr

Proposed by Chris Biersbach (OpenERP)
Status: Merged
Merged at revision: 2390
Proposed branch: lp:~openerp-dev/openerp-web/mck-opw-582226-cbi
Merge into: lp:~openerp-dev/openerp-web/6.1-web-custo-tfr
Diff against target: 16 lines (+5/-1)
1 file modified
addons/web/static/src/js/search.js (+5/-1)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/mck-opw-582226-cbi
Reviewer Review Type Date Requested Status
OpenERP R&D Team Pending
Review via email: mp+141707@code.launchpad.net

Description of the change

Clicking on "add advanced filter" in IE8 now correctly works and makes using advanced filters possible.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/web/static/src/js/search.js'
--- addons/web/static/src/js/search.js 2012-06-29 12:18:05 +0000
+++ addons/web/static/src/js/search.js 2013-01-03 07:37:21 +0000
@@ -192,7 +192,11 @@
192 filters: result,192 filters: result,
193 disabled_filter_message: _t('Filter disabled due to invalid syntax')193 disabled_filter_message: _t('Filter disabled due to invalid syntax')
194 }));194 }));
195 filters.change(self.on_filters_management);195 //Bind onchange without using jQuery due to issue in IE8 with jQuery 1.6.4 http://bugs.jquery.com/ticket/6593
196 //Due to which change is called twice because we are changing the selection index from on_cahnge itself
197 filters[0].onchange = function(e){
198 self.on_filters_management(e);
199 }
196 });200 });
197 },201 },
198 /**202 /**

Subscribers

People subscribed via source and target branches

to all changes: