Launchpad itself

Code review comment for lp:~kfogel/launchpad/old-1.6-ml-archiver-ui

Revision history for this message

Martin Albisetti (beuno) wrote on 2009-04-29:

Hi Karl,

Thanks for working on this, it's a very big improvement.

The only comment I have, is that the sender and the date should be somehow visually distinct (looking at: http://www.red-bean.com/kfogel/ml-archiver-ui-screenshots/new-ml-archives-by-thread-top.png), maybe the sender could be in bold?
Also, maybe the date should be YYYY-MM-DD instead of YYYY/MM/DD, as using slashes usually means it's formatted.

Neither of these are blockers for the branch to land, but would be nice :)

review: Approve (ui)