Closed Bug 671236 Opened 10 years ago Closed 10 years ago
Flatter appearance for quick filter bar
110.70 KB, image/png
17.85 KB, image/png
25.63 KB, image/png
25.00 KB, image/png
52.50 KB, image/png
3.06 KB, image/png
10.66 KB, patch
|Details | Diff | Splinter Review|
The quick filter bar sticks out a bit visually from the rest of the Aero theme. Lets look into how we can solve this.
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!
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)
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?
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.
Patch with icons from toolbars. Still needing icons for Pin and Star.
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-
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?
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.
Here are the new icons. Hope this looks better.
Attachment #546754 - Attachment is obsolete: true
Patch with updated icons.
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+
I renamed the image to quick-search-icons-aero.png
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 8.0
You need to log in before you can comment on or make changes to this bug.