← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~rharding/launchpad/info_portlet_1052551 into lp:launchpad

 

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.


Follow ups

References