1187317 - redundant notification

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System::Status bar, Utility tray, Notification, defect)

Description

[Natalia Martinez-Winter]

User-Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0

The first notification (with number of notifications) is useless and misleading. Made me believe it's clickable but nothing happens when you click on it. Takes notification screen space for nothing. Sometimes it even shows "zero notification"!

Comment 1

[Natalia Martinez-Winter]

Attachment: screenshot.png

Comment 2

[Peter Dolanjski [:pdol]]

UX to look at for future design or a revision.

Comment 3

[Tiffanie Shakespeare [:tif]]

Passing NI to Eric as he's taking a look at notifications.

Thanks for pinging the UX team!

Comment 4

[Eric Pang [:epang] UX]

I've never seen this notification, is it an official notification from Mozilla?  If, so do we know why it's there?  If not, where is it from?  Seems pretty pointless if tapping it goes nowhere :)

Leaving need info on myself to keep track of this bug.

Comment 5

[Natalia Martinez-Winter]

I think the misleading part comes from the blue icon. "blue" color is often used for clickable things. This line is part of the utility tray.

Comment 6

[Eric Pang [:epang] UX]

Hi Natalia, thanks for getting back to me.  Sorry, I completely misunderstood this before!  What we can do is make the icon white to help with any confusion about being tappable.

Guillaume, is this something you can help with?  I can't remember if the notification icon in the header was an svg, if so can we update it to be #ffffff?

Also can center the icon between the left edge and text?

let me know if you can help, thanks!

Comment 7

[GitHub Autolander Bot]

Attachment: [gaia] gmarty:Bug-1187317-Redundant-notification > mozilla-b2g:master

Comment 8

[Guillaume Marty [:gmarty]]

Comment on attachment 8650470 [details] [review]
[gaia] gmarty:Bug-1187317-Redundant-notification > mozilla-b2g:master

I removed the color on the notification and moved it 3 pixels on the left to be aligned with the other icons on the notification.
Alberto, can you take a look?

Comment 9

[Alberto Pastor [:albertopq]]

Comment on attachment 8650470 [details] [review]
[gaia] gmarty:Bug-1187317-Redundant-notification > mozilla-b2g:master

The code looks good to me. Eric, can you check if that's what we visually expect?

Thanks!

Comment 10

[Eric Pang [:epang] UX]

Comment on attachment 8650470 [details] [review]
[gaia] gmarty:Bug-1187317-Redundant-notification > mozilla-b2g:master

sorry for they delay on reviewing this. Looks good, but looks like there's been other changes to the notification tray.  For example, now when there's no notification the header reads "No Notifications" instead of "0 Notifications" so we just need to make sure to lose newer changes.

Thanks Guillaume!

Comment 11

[Tiffanie Shakespeare [:tif]]

I think Eric means "to NOT lose newer changes". :)

Comment 12

[Eric Pang [:epang] UX]

(In reply to Tiffanie Shakespeare [:tif] UX from comment #11)
> I think Eric means "to NOT lose newer changes". :)

yes!!  Thanks Tiff :)

Comment 13

[Carsten Book [:Tomcat]]

https://github.com/mozilla-b2g/gaia/commit/81b7d1916c750abb00bf52628701d7889d8fdd32