853531 - Use a nicer icon for the new "New Mail" notification.

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Richard Marti (:Paenglab)]

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)]

Attachment: patch

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)]

Attachment: screenshot

Screenshot for easier ui-r

Comment 3

[Richard Marti (:Paenglab)]

Attachment: 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)]

Slightly skeptical of the tilting, but I like the flat colors.
How would it look straight on?

Comment 5

[Richard Marti (:Paenglab)]

Attachment: 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.

Comment 6

[Richard Marti (:Paenglab)]

Attachment: screenshot with application icons

This screenshot shows the notification with the three application icons.

Comment 7

[Richard Marti (:Paenglab)]

A try build is available here: http://ftp.mozilla.org/pub/mozilla.org/thunderbird/try-builds/[e-mail]-1847e11e8f95

Comment 8

[Andreas Nilsson (:andreasn)]

Looks great!

Comment 9

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 738905 [details] [diff] [review]
Use the application icons

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

Thanks,
Blake.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/1ae201906fed