Closed Bug 402388 Opened 12 years ago Closed 12 years ago

Fix vertical alignment of headers in report details

Categories

(Other Applications Graveyard :: Reporter, defect, minor)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9beta1

People

(Reporter: raccettura, Assigned: raccettura)

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
Vertical alignment should really be top in report details.  On windows this isn't such a big deal, but on Mac OS X, the build info can be rather long, and with the header in the middle it looks pretty strange.
Comment on attachment 287271 [details] [diff] [review]
Patch

This seems to do the trick.

Should note also fixed some spacing to make it more consistent.
Attachment #287271 - Flags: review?(dao)
Comment on attachment 287271 [details] [diff] [review]
Patch

Note that I'm not a peer of any module, but this is certainly the right thing to do.
Attachment #287271 - Flags: review?(dao) → review+
Comment on attachment 287271 [details] [diff] [review]
Patch

There aren't really any peers... kinda just piggybacks on browser/toolkit.  I figure if you could manage bug 389131... good enough for something so simple ;-)

requesting M9 approval, it's just minor polish to pretty things up.
Attachment #287271 - Flags: approvalM9?
Comment on attachment 287271 [details] [diff] [review]
Patch

a=endgame drivers for M9
Attachment #287271 - Flags: approvalM9?
Attachment #287271 - Flags: approvalM9+
Attachment #287271 - Flags: approval1.9+
Checking in extensions/reporter/resources/skin/classic/reporter/reportResults.css;
/cvsroot/mozilla/extensions/reporter/resources/skin/classic/reporter/reportResults.css,v  <--  reportResults.css
new revision: 1.2; previous revision: 1.1
done
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M9
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.