1400159 - Notification area icon uses old shape

Status: RESOLVED FIXED

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

Description

[Michael Monreal [:micmon]]

Attachment: Screenshot_20170904-213413.png

Firefox for Android (Nightly) still uses the old notification icon. As the shape of the new app icon is different, this icon should also be changed. I don't know hat kind of icon (format/size/filename) is needed here but at least a matching symbolic icon which can be adapted already exists, see bug #1387698.

Comment 1

[Chenxia Liu [:liuche]]

Hi Jing-wei, looks like this icon wasn't updated with bug 1366669.

Comment 2

[Jing-wei Wu [:jwu]]

Thanks for notification, we would update the notification icon ASAP.

Comment 3

[Jing-wei Wu [:jwu]]

Attachment: Bug 1400159 - Part 1: Update the notification icons.

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

Comment 4

[Jing-wei Wu [:jwu]]

Attachment: Bug 1400159 - Part 2: Update the QR code icons.

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

Comment 5

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8909621 [details]
Bug 1400159 - Part 2: Update the QR code icons.

https://reviewboard.mozilla.org/r/181100/#review186314

Comment 6

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8909620 [details]
Bug 1400159 - Part 1: Update the notification icons.

https://reviewboard.mozilla.org/r/181098/#review186316

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2ed53c28b7d1
Part 1: Update the notification icons. r=nechen
https://hg.mozilla.org/integration/autoland/rev/e7c351009e60
Part 2: Update the QR code icons. r=nechen

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2ed53c28b7d1
https://hg.mozilla.org/mozilla-central/rev/e7c351009e60

Comment 9

[Oana Horvath [:ohorvath]]

Verified as fixed on nightly (2017-09-20). The Nightly notifications and QR code icons are updated. 
Devices:
Huawei MediaPad M2 (Android 5.1.1)
Asus ZenPad 8.0 Z380KL (Android 6.0.1)
Prestigio Grace X5 (Android 4.4.2)
Motorola Nexus 6 (Android 7.0)

Comment 10

[Michael Monreal [:micmon]]

The old icon is still used even on F58 for the video player notification.

Comment 11

[Ian Moody [:Kwan] (UTC+1)]

As comment 10 notes, this isn't fixed:

STR 1:
1) Play a video on youtube

STR 2:
2) Send a tab from computer to phone

Comment 12

[Nevin Chen(Not active on Bugzilla)]

Thanks. 
I'll use bug 1421859 to fix this.