launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #13411
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