Vaibhav (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/trunk-bug-918190-vda into lp:openerp-web.

Requested reviews:
  OpenERP R&D Web Team (openerp-dev-web)
Related bugs:
  Bug #918190 in OpenERP Web: "[6.1.rc1] Mandatory field stay red after invalid 
selection"
  https://bugs.launchpad.net/openerp-web/+bug/918190

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-918190-vda/+merge/93375

Required element:toggle invalid/valid on valid value.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-918190-vda/+merge/93375
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-bug-918190-vda.
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2012-02-15 12:39:01 +0000
+++ addons/web/static/src/js/view_form.js	2012-02-16 11:09:20 +0000
@@ -1408,7 +1408,7 @@
         if (!this.disable_utility_classes) {
             this.$element.toggleClass('disabled', this.readonly);
             this.$element.toggleClass('required', this.required);
-            if (show_invalid) {
+            if (show_invalid || !this.invalid) {
                 this.$element.toggleClass('invalid', !this.is_valid());
             }
         }

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to