Closed Bug 671236 Opened 10 years ago Closed 10 years ago

Flatter appearance for quick filter bar

Categories

(Thunderbird :: Theme, defect)

x86
Windows Vista
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 8.0

People

(Reporter: andreasn, Assigned: Paenglab)

References

Details

Attachments

(7 files, 6 obsolete files)

The quick filter bar sticks out a bit visually from the rest of the Aero theme.
Lets look into how we can solve this.
Attached image mockup
Here is a mockup for how it could look:
* Same color as message header background.
* Monochrome icons like the rest of the interface (these could gain color when active, somewhat like https://bug668336.bugzilla.mozilla.org/attachment.cgi?id=544097 in bug #668336 but possibly only one color)
* Also note the list headers from bug #671123
Maybe the buttons should look like the toolbar buttons (i.e. be monochome, but have some shading so they stand out a bit more). Right now, the buttons seem more like indicator icons than actual buttons that will do something...
Yes, that makes sense!
Attached patch QFB-Patch (obsolete) — Splinter Review
This patch gives the same background color as the message header. The buttons have the same appearance as the toolbarbuttons.
I used the svg images from Bug 588007 where I could. The sticky button is now only using the monochrome icon.

This patch contains now only the css. If ui-r is okay I'll add also the removing of the unneeded icons.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #546404 - Flags: ui-review?(nisses.mail)
Attached image Patch in action
Attached patch QFB-Patch (obsolete) — Splinter Review
I experimented with buttons under Classic and High contrast themes. For me it looks good also under Classic and High contrast. Only the tree icons for Pin, Contact and Tag are looking not so good under High contrast.

:andreasn could you make SVG images for these tree buttons?
Attachment #546404 - Attachment is obsolete: true
Attachment #546409 - Flags: ui-review?(nisses.mail)
Attachment #546404 - Flags: ui-review?(nisses.mail)
I think the buttons are *less* flat with this patch, rather than more. I'm not convinced that SVGs are the right way to go here either (unless we decided to make all the button icons SVGs, of course), since every button icon in Thunderbird uses the shaded/inset appearance from Firefox, and we should probably stay consistent here.
Attached patch QFB-Patch (obsolete) — Splinter Review
Patch with icons from toolbars. Still needing icons for Pin and Star.
Attachment #546409 - Attachment is obsolete: true
Attachment #546433 - Flags: ui-review?(nisses.mail)
Attachment #546409 - Flags: ui-review?(nisses.mail)
Attached image new icons (obsolete) —
Here are more sunken in, and smaller icons.
This also uses color for the active state of the pin (this makes me think we might want to make #668336 INVALID)
Comment on attachment 546433 [details] [diff] [review]
QFB-Patch

setting ui-r minus as we want to use other icons. I also think it would be better to keep the button look on hover and pressed only.
Attachment #546433 - Flags: ui-review?(nisses.mail) → ui-review-
Attached patch QFB-Patch V2 (obsolete) — Splinter Review
Patch with button look only on hover and checked. This patch is also using the new icon image but this isn't in the patch because I can't create a binary patch. Andreas, please can you create a patch with the image file?

The Tag image looks now a little bit bigger than the others (the bottom is a little bit lower than the others, 18px to 16px). Is this okay or would you add a icon to quick-search-icons.png?
Attachment #546433 - Attachment is obsolete: true
Attachment #546878 - Flags: ui-review?(nisses.mail)
Yeah, you're right, it looks a bit off. Updated version coming up!
Andreas you can now only attach the updated icons to this bug. I'm now able to create patches with binary diffs.
Attached image Even newer icons
Here are the new icons. Hope this looks better.
Attachment #546754 - Attachment is obsolete: true
Attached patch QFB-Patch v3 (obsolete) — Splinter Review
Patch with updated icons.
Attachment #546878 - Attachment is obsolete: true
Attachment #547936 - Flags: ui-review?(nisses.mail)
Attachment #546878 - Flags: ui-review?(nisses.mail)
Comment on attachment 547936 [details] [diff] [review]
QFB-Patch v3

Looks much better now! Good call about the tag icon size.
Attachment #547936 - Flags: ui-review?(nisses.mail) → ui-review+
Comment on attachment 547936 [details] [diff] [review]
QFB-Patch v3

:squib is it okay to ask you for review?
Attachment #547936 - Flags: review?(squibblyflabbetydoo)
(In reply to comment #20)
> Comment on attachment 547936 [details] [diff] [review] [review]
> QFB-Patch v3
> 
> :squib is it okay to ask you for review?

I can review this. In general, it's a bit tougher for me to review Windows-only patches, since I don't build on Windows, but it shouldn't be a big problem here. That said, feel free to ask me for review for any of the Linux theme patches that you're working on, since those are easy for me to test. :)
Comment on attachment 547936 [details] [diff] [review]
QFB-Patch v3

Review of attachment 547936 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me, but I think quick-search-icons.png should be renamed to quick-search-icons-aero.png, since we might want to make quick-search-icons.png for the XP theme too. r=me with that fixed.

I'm also a little unsure about the visuals for the pressed button state, but since that's how Firefox handles its buttons too, I'm not going to worry about it. :)
Attachment #547936 - Flags: review?(squibblyflabbetydoo) → review+
Blocks: 645294
I renamed the image to quick-search-icons-aero.png
Attachment #547936 - Attachment is obsolete: true
Attachment #548800 - Flags: ui-review+
Attachment #548800 - Flags: review+
Keywords: checkin-needed
http://hg.mozilla.org/comm-central/rev/0fd8d928a349
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 8.0
You need to log in before you can comment on or make changes to this bug.