Last Comment Bug 853531 - Use a nicer icon for the new "New Mail" notification.
: Use a nicer icon for the new "New Mail" notification.
Product: Thunderbird
Classification: Client Software
Component: Theme (show other bugs)
: unspecified
: All All
: -- normal (vote)
: Thunderbird 23.0
Assigned To: Richard Marti (:Paenglab)
Depends on:
Blocks: 856759
  Show dependency treegraph
Reported: 2013-03-21 10:58 PDT by Richard Marti (:Paenglab)
Modified: 2013-05-13 06:40 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

patch (2.43 KB, patch)
2013-03-21 11:05 PDT, Richard Marti (:Paenglab)
no flags Details | Diff | Splinter Review
screenshot (10.62 KB, image/png)
2013-03-21 11:06 PDT, Richard Marti (:Paenglab)
no flags Details
SVG file (3.75 KB, image/svg+xml)
2013-03-21 11:12 PDT, Richard Marti (:Paenglab)
no flags Details
Use the application icons (11.04 KB, patch)
2013-04-17 23:16 PDT, Richard Marti (:Paenglab)
bwinton: review+
bwinton: ui‑review+
Details | Diff | Splinter Review
screenshot with application icons (39.54 KB, image/png)
2013-04-17 23:18 PDT, Richard Marti (:Paenglab)
no flags Details

Description Richard Marti (:Paenglab) 2013-03-21 10:58:54 PDT
On OS X TB is now using the new notifications. The actual icon is looking a little bit out of place.

This bug is to replace it with a better looking one.
Comment 1 Richard Marti (:Paenglab) 2013-03-21 11:05:23 PDT
Created attachment 727780 [details] [diff] [review]

Nicer icon

Andreas, I took you because bwinton liked the new icon already and he wants someone without bias to review it :)
Comment 2 Richard Marti (:Paenglab) 2013-03-21 11:06:16 PDT
Created attachment 727781 [details]

Screenshot for easier ui-r
Comment 3 Richard Marti (:Paenglab) 2013-03-21 11:12:17 PDT
Created attachment 727785 [details]
SVG file

SVG file for the repository.

Or if you think it needs some tweaking, you have the file to change it.
Comment 4 Andreas Nilsson (:andreasn) 2013-04-16 13:13:43 PDT
Slightly skeptical of the tilting, but I like the flat colors.
How would it look straight on?
Comment 5 Richard Marti (:Paenglab) 2013-04-17 23:16:41 PDT
Created attachment 738905 [details] [diff] [review]
Use the application icons

Over IRC we decided to use the application icons like it's system standard.

This patch copies the branding icon to new-mail-alert.png which is then used for the toolkit notification.
I copy the icon in branding because I see no way how to reference the icon in theme For nightly it's in /mail/branding/nightly, for Aurora in /mail/branding/aurora and for Release (and Beta?) it's in /other-licenses/branding/thunderbird. If it would be always on the same position during build I could use this position.
Comment 6 Richard Marti (:Paenglab) 2013-04-17 23:18:05 PDT
Created attachment 738906 [details]
screenshot with application icons

This screenshot shows the notification with the three application icons.
Comment 7 Richard Marti (:Paenglab) 2013-04-17 23:20:55 PDT
A try build is available here:
Comment 8 Andreas Nilsson (:andreasn) 2013-04-19 08:30:54 PDT
Looks great!
Comment 9 Blake Winton (:bwinton) (:☕️) 2013-05-12 18:49:25 PDT
Comment on attachment 738905 [details] [diff] [review]
Use the application icons

Looks good!  r=me, ui-r=me.

Comment 10 Ryan VanderMeulen [:RyanVM] 2013-05-13 06:39:48 PDT

Note You need to log in before you can comment on or make changes to this bug.