Review: Needs Fixing

1. Why is the *exact same CSS* twice in the file on the same class?
2. Use CSS properties when they're available, gods, why set padding-top, 
padding-right and padding-left separately when a single `padding` property will 
do?
3. The un-prefixed `border-radius` will not work in some older browsers (mainly 
Firefox 3.6), please use the `radius` mixin defined at the top of the file (you 
can see how it's used line 74 or base.sass)
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-need_action-tde/+merge/100797
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-need_action-tde.

_______________________________________________
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