Review: Approve code Looks good. A couple of minor points:
It would be nice to fix the long line on line 26 of the diff. Would it be possible to put the explicit style attributes on line 31 and 79 into a stylesheet? -- https://code.launchpad.net/~rharding/launchpad/info_portlet_1052551/+merge/129706 Your team Launchpad code reviewers is subscribed to branch lp:launchpad. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
