Code review comment for lp:~openerp-dev/openerp-web/7.0-bug-1097738-bth

Revision history for this message
Bhumi Thakkar (Open ERP) (bth-openerp) wrote :

Improve code as per your suggestion.

> Could you put your declaration in the same block as the already existant
> oe_form_required declaration ?
>
> Eg:
> .oe_form_required
> input:not([disabled]):not([readonly]),
> select:not([disabled]):not([readonly]),
> textarea:not([disabled]):not([readonly]), &.oe_form_required iframe
> background-color: #D2D2FF !important
>
> While you are at it, could you put a saas variable for the required color
> #D2D2FF (also used in editable list)

review: Needs Resubmitting

« Back to merge proposal