Closed Bug 691730 Opened 13 years ago Closed 13 years ago

toolbar buttons and global search box are different height

Categories

(Thunderbird :: Theme, defect)

All
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 10.0

People

(Reporter: andreasn, Assigned: Paenglab)

References

(Blocks 1 open bug)

Details

Attachments

(3 files, 1 obsolete file)

Attached image mockup
The search box is 2px shorter than the toolbar buttons under Aero.
Blocks: 667235
Attached patch Make the min-height 24px (obsolete) — Splinter Review
I've changed the min-height from 22px to 24px and gave a top and bottom margin of 1px instead of 2px to avoid a growing of the toolbar. I made the same change to the Quick Filter search box to have the same height of the search boxes.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #564644 - Flags: ui-review?(nisses.mail)
Attachment #564644 - Flags: review?(nisses.mail)
Comment on attachment 564644 [details] [diff] [review]
Make the min-height 24px

The padding for filters feels a bit too narrow now.
Attachment #564644 - Flags: ui-review?(nisses.mail) → ui-review-
Attached image filter padding mockups
Here is mockups with padding 1px, 2px and 3px.
I'm currently leaning towards 3px padding, as that would fit the smaller icons in this bar (it almost aligns with the top and bottom of the pin) and show the difference between the search bars a bit more. This is also the height of the Find bar search box in Firefox.
With this patch I change only the Global search box.

The default margin for the QFB search box is 2px and the actual height of this box is 23px (uneven top- and bottom-padding in toolkit). Add 1px on top and bottom would make the QFBar 2px taller.
Attachment #564644 - Attachment is obsolete: true
Attachment #564644 - Flags: review?(nisses.mail)
Attachment #565202 - Flags: ui-review?(nisses.mail)
Attachment #565202 - Flags: review?(nisses.mail)
Comment on attachment 565202 [details] [diff] [review]
Make the min-height 24px v2

As discussed on IRC, skipping the QFB part of this patch as the sizing of it proved to be slightly tricky. Might happen as another bug.
This looks good ui and code-wise, good to go!
Attachment #565202 - Flags: ui-review?(nisses.mail)
Attachment #565202 - Flags: ui-review+
Attachment #565202 - Flags: review?(nisses.mail)
Attachment #565202 - Flags: review+
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/f9887fb58929
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 10.0
You need to log in before you can comment on or make changes to this bug.