Compared to Firefox, the buttons on the tab bar are slightly too small: 1) There's a top margin that makes them shorter than they should be 2) There's a 1px gap between the bottom of the hover gradient and the tab bar (see attached screenshot)
This patch removes the 3px top-margins. Additionally I made the tabs the same height as Firefox (27px instead of 26px) and cleaned the files from remnants of pre tabs-on-top time. I moved the #print-preview-toolbar definition from primaryToolbar-aero.css to messenger-aero.css because it wasn't used there. On every file I touched I removed the 'all and' from the media queries (see Bug 713822).
Comment on attachment 592489 [details] [diff] [review] patch Excellent!
Attachment #592489 - Flags: ui-review?(nisses.mail) → ui-review+
Attachment #592489 - Flags: review?(squibblyflabbetydoo) → review?(nisses.mail)
Comment on attachment 592489 [details] [diff] [review] patch And code looks good too.
Attachment #592489 - Flags: review?(nisses.mail) → review+
Comment on attachment 592489 [details] [diff] [review] patch [Approval Request Comment] Regression caused by (bug #): 715495 which is in TB 12 User impact if declined: No Testing completed (on c-c, etc.): No testing, only review Risk to taking this patch (and alternatives if risky): Should have no risk, it's only css
Attachment #592489 - Flags: approval-comm-aurora?
Attachment #592489 - Flags: approval-comm-aurora? → approval-comm-aurora+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
You need to log in before you can comment on or make changes to this bug.