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.
Created attachment 727780 [details] [diff] [review] patch Nicer icon Andreas, I took you because bwinton liked the new icon already and he wants someone without bias to review it :)
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.
Slightly skeptical of the tilting, but I like the flat colors. How would it look straight on?
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 jar.mn because I see no way how to reference the icon in theme jar.mn. 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.
Created attachment 738906 [details] screenshot with application icons This screenshot shows the notification with the three application icons.
A try build is available here: http://firstname.lastname@example.org
Comment on attachment 738905 [details] [diff] [review] Use the application icons Looks good! r=me, ui-r=me. Thanks, Blake.