907976 - Comment header should have default line-height, vertical-align set

Status: RESOLVED FIXED

(bugzilla.mozilla.org Graveyard :: Sandstone/Mozilla Skin, defect)

Description

[Matt Basta [:basta]]

The `.bz_comment_head, .bz_first_comment_head {}` declaration in global.css would look significantly better if it had a default line-height which matched the size of the contents (32px). The following changes should make things look a bit nicer:

.bz_comment_head, .bz_first_comment_head {
    font-size: 15px;
    line-height: 32px;
}
.bz_comment_head img, .bz_first_comment_head img {
    margin-right: 0.25em;
    vertical-align: middle;
}

Before: http://cl.ly/image/3E1106353N3J
After: http://cl.ly/image/143Q1p2H1f3l

The margin-right on the img elements is half of what the spans in the headers get, so it matches a bit better. The font-size bumps the sizing up just a bit to avoid having a big image and itty-bitty text next to it.

Comment 1

[David Lawrence [:dkl]]

It does look better. Thanks!

Committing to: bzr+ssh://[e-mail]/bmo/4.2
modified skins/contrib/Mozilla/global.css
Committed revision 8964.

dkl