Closed Bug 402388 Opened 17 years ago Closed 17 years ago

Fix vertical alignment of headers in report details

Categories

(Other Applications Graveyard :: Reporter, defect)

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: 17 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.

Attachment

General

Created:
Updated:
Size: