Chris Biersbach (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/mck-opw-582226-cbi into 
lp:~openerp-dev/openerp-web/6.1-web-custo-tfr.

Requested reviews:
  OpenERP R&D Team (openerp-dev)

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/mck-opw-582226-cbi/+merge/141707

Clicking on "add advanced filter" in IE8 now correctly works and makes using 
advanced filters possible.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/mck-opw-582226-cbi/+merge/141707
Your team OpenERP R&D Team is requested to review the proposed merge of 
lp:~openerp-dev/openerp-web/mck-opw-582226-cbi into 
lp:~openerp-dev/openerp-web/6.1-web-custo-tfr.
=== 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 @@
                 filters: result,
                 disabled_filter_message: _t('Filter disabled due to invalid syntax')
             }));
-            filters.change(self.on_filters_management);
+            //Bind onchange without using jQuery due to issue in IE8 with jQuery 1.6.4 http://bugs.jquery.com/ticket/6593
+            //Due to which change is called twice because we are changing the selection index from on_cahnge itself
+            filters[0].onchange = function(e){
+                self.on_filters_management(e);
+            }
         });
     },
     /**

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : openerp-dev-gtk@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to