code review comment font is too big

[Martin Pool]

The monospace comment font is larger than the comment font used for eg bugs, and roughly twice as big as the font used for the headers like "Status". As a result if your base font is set to a reasonable size, the comment font looks far too big. (jml correctly observes that you can work around this by scaling the page so the other fonts are too small. :-)

It should be the same font across launchpad.

[Martin Pool (mbp)]

merge proposal screenshot

[Martin Pool (mbp)]

bug screenshot

[Julian Edwards (julian-edwards)]

After mpt duped my bug 344769 to this one, is this a Konqueror problem only?

[Matthew Paul Thomas (mpt)]

Well, it might be worse currently in Konqueror than in other browsers because Konqueror's default font settings are different. But if code review comments become styled in exactly the same way as bug comments (i.e. using the same rule in the style sheet as bug comments do), then it will be just as fixed in Konqueror as it is in other browsers.

[Martin Pool (mbp)]

2009/3/19 Matthew Paul Thomas <email address hidden>:
> Well, it might be worse currently in Konqueror than in other browsers
> because Konqueror's default font settings are different. But if code
> review comments become styled in exactly the same way as bug comments
> (i.e. using the same rule in the style sheet as bug comments do), then
> it will be just as fixed in Konqueror as it is in other browsers.

I'm pretty sure they are not styled the same as bug comments, because
I can reproduce this in a plain firefox install. (I haven't traced
through the html/css to work out just why.) That's the bug. It's not
browser specific.

--
Martin <http://launchpad.net/~mbp/>

[Tim Penhey (thumper)]

Found it.

[Tim Penhey (thumper)]

Missed the RC cut off.

[Tim Penhey (thumper)]

Fixed in RF 8712.

[Martin Pool (mbp)]

Reopening: it's fixed in the main code review page but not in the "reply" page, eg https://code.edge.launchpad.net/~mbp/bzr/340347-log-decorator/+merge/8075/comments/20848/+reply

[Tim Penhey (thumper)]

Pushing off to 2.2.8 for the comment page itself.

[Tim Penhey (thumper)]

Fixed in RF 8986.