Review: Needs Fixing

1. Indentation is broken, please fix it and configure your text editor 
correctly: indentation should be 4 spaces per indent across the project, no 
tabs.

2. I think extracting (at least) the inner part of the pager and calling it 
from both top and bottom would be a better idea than duplicating that code, and 
potentially forgetting to edit both sections in case of requirement changes.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-878106-tpa/+merge/80026
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-bug-878106-tpa.

_______________________________________________
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