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

