1241824 - The notification icon (ic_status_logo) looks blurry/pixelated

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

Description

[Sebastian Kaspari (:sebastian; :pocmo)]

Especially on the N6P the notification icon looks blurry/pixelated compared to other system notification icons.

It looks like we only have hdpi and xhdpi versions of the icon:
https://dxr.mozilla.org/mozilla-central/search?q=pathic_status_logo&redirect=false&case=false

Should we add a xxhdpi/xxxhdpi version?

In general the xhdpi version looks pixelated - is this just because of the size (48x48) or could we improve the quality somehow?
https://dxr.mozilla.org/mozilla-central/raw/mobile/android/base/resources/drawable-xhdpi-v11/ic_status_logo.png

Comment 1

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Attachment: notification_fox.zip

A quick search lead me here: http://iconhandbook.co.uk/reference/chart/android/

So I attached some assets according to this doc. Let me know how these work!

Comment 2

[Sebastian Kaspari (:sebastian; :pocmo)]

With the xxhdpi version the icon is a bit more crisp.

I think some of the feeling that the icon is blurry comes from the fur of the fox. In this small size this makes the icon look rough and not "round".

Comment 3

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: MozReview Request: Bug 1241824 - Add xxhdpi version of notification icon. r?ahunt

Review commit: https://reviewboard.mozilla.org/r/34491/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/34491/

Comment 4

[Sebastian Kaspari (:sebastian; :pocmo)]

I've been running PNGcrush on the icon to reduce its already small size a bit more:
-rw-r--r--   1 sebastian  staff  2243 Jan 26 01:28 ic_status_logo.png
-rw-r--r--   1 sebastian  staff  1343 Feb 11 11:50 ic_status_logo_pngcrush.png

Comment 5

[Andrzej Hunt :ahunt]

Comment on attachment 8718287 [details]
MozReview Request: Bug 1241824 - Add xxhdpi version of notification icon. r?ahunt

https://reviewboard.mozilla.org/r/34491/#review31419

Comment 6

[Sebastian Kaspari (:sebastian; :pocmo)]

https://hg.mozilla.org/integration/fx-team/rev/ba082b772025e07b36f900f36c07b3b34c050762
Bug 1241824 - Add xxhdpi version of notification icon. r=ahunt

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ba082b772025