Closed Bug 907976 Opened 6 years ago Closed 6 years ago

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

Categories

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

Production
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: basta, Assigned: dkl)

Details

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.
It does look better. Thanks!

Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.2
modified skins/contrib/Mozilla/global.css
Committed revision 8964.

dkl
Assignee: nobody → dkl
Status: NEW → RESOLVED
Closed: 6 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Product: bugzilla.mozilla.org → bugzilla.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.