Chirag Patel (OpenERP) has proposed merging
lp:~openerp-dev/openerp-web/6.1-opw-579322-cpa into lp:openerp-web/6.1.
Requested reviews:
OpenERP Core Team (openerp)
For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-579322-cpa/+merge/124918
Hello,
Demo:
1) Edit user preferences.
2) In language combo box select the empty line.
> Field becomes red.
3) Return to English.
Observed: Field stay with "Red" color.
Expected: Field color should be "Alpine Blue".
Thanks.
--
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-579322-cpa/+merge/124918
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openerp-web/6.1-opw-579322-cpa.
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js 2012-09-03 13:17:44 +0000
+++ addons/web/static/src/js/view_form.js 2012-09-18 13:19:23 +0000
@@ -1467,7 +1467,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.$element.hasClass('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
