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
1=== modified file 'addons/web/static/src/js/search.js'
2--- addons/web/static/src/js/search.js 2012-06-29 12:18:05 +0000
3+++ addons/web/static/src/js/search.js 2013-01-03 07:37:21 +0000
4@@ -192,7 +192,11 @@
5 filters: result,
6 disabled_filter_message: _t('Filter disabled due to invalid syntax')
7 }));
8- filters.change(self.on_filters_management);
9+ //Bind onchange without using jQuery due to issue in IE8 with jQuery 1.6.4 http://bugs.jquery.com/ticket/6593
10+ //Due to which change is called twice because we are changing the selection index from on_cahnge itself
11+ filters[0].onchange = function(e){
12+ self.on_filters_management(e);
13+ }
14 });
15 },
16 /**

Subscribers

People subscribed via source and target branches

to all changes: