1153500 - Get a new icon for the Notification permission prompt

Status: RESOLVED DUPLICATE of bug 1192458

(Core :: DOM: Push Subscriptions, defect)

Comment 1

[Stephen Horlander [:shorlander]]

Attachment: icon-notificationsTray-push.svg

Notification Icon

States Left-to-Right: Normal, Hover, Active

Comment 2

[Stephen Horlander [:shorlander]]

Attachment: icon-panel-push.svg

Panel Icon

Comment 3

[Stephen Horlander [:shorlander]]

Overview / Mockups:

http://people.mozilla.org/~shorlander/mockups/Push-Notifications-Icon/Push-Notification-Icons.html

Comment 5

[Doug Turner (:dougt)]

I think I want these in png, like the other ones.

  skin/classic/browser/Geolocation-16.png
  skin/classic/browser/[e-mail]
  skin/classic/browser/Geolocation-64.png
  skin/classic/browser/[e-mail]
  skin/classic/browser/Push-16.png
  skin/classic/browser/[e-mail]
  skin/classic/browser/Push-64.png
  skin/classic/browser/[e-mail]

Dolske, what do we use now?

Comment 6

[Stephen Horlander [:shorlander]]

We have been replacing PNGs with SVG where possible. Just updated the add-ons install icons with SVG in bug 1162584

Comment 7

[Justin Dolske [:Dolske]]

Yep, SVG is the new hotness. Gets you sharp images at both normal and OS X Retina resolutions, as well as handing all the oddball scaling factors on Windows, and all in just one file.

Comment 8

[Doug Turner (:dougt)]

Attachment: Screenshot 2015-05-31 23.04.19.png

Comment 9

[Doug Turner (:dougt)]

Attachment: Screenshot 2015-05-31 23.07.21.png

Comment 10

[Doug Turner (:dougt)]

The above screenshot is for "Notifications".

Comment 11

[Doug Turner (:dougt)]

Attachment: Screenshot 2015-05-31 23.07.30.png

Comment 12

[Doug Turner (:dougt)]

The above screenshot is for Push notifications.

I do not think I am happy with how this ended up (ignore fact that the smaller notification icon has 3 icons in it -- not just one).  Basically the notification and push look very similar and they will probably be used in conjunction with each other.

Comment 13

[Doug Turner (:dougt)]

shorlander, if you like to discuss this week in person/vidyo, let me know.

Comment 14

[Doug Turner (:dougt)]

https://github.com/dougt/gecko-dev/tree/dougt_bug_1153500_pretty_icon

Comment 15

[Madhava Enros [:madhava]]

Hi Doug - 

Is your concern that the concept/metaphor for the Push Notifications icon(s) is similar to that for Notifications but shouldn't be? I.e. do you they think they represent concepts that should be portrayed as more dissimilar? Or is it just that it's odd to have two icons that differ only very slightly that a user may encounter near each other in a task flow?

If it's the latter -- we could, in the sort term, just use the same new (better!) iconography in both cases and rely on the text in the question to make a difference clear to the user. If it's the former -- this is an issue we've been struggling with here for a while: how would you characterize the difference in how these two permissions will likely be used? I know you mentioned that Push Notifications can be used for more things like updating data in a web app in the background, but I thought we'd established that that was not the principal use, in the short term, anyway.


(On the three icons not one issue - you mean ignore it because the sprite's just not being used properly (i.e. all three states shown at the same time), right? So you're flagging that that's NOT the issue you're pointing out?)

Comment 16

[Doug Turner (:dougt)]

It is odd to have two icons so similar for things that can have very different use cases.  We can use what we have in the short term, but I do not think that is great.  It might be that we join the permissions for web notifications and push.  We should work through these user experiences and try to understand what is best.

Can we do this for 41, or do you prefer that we go ahead and use these icons (instead of geo!) and follow up?

Comment 17

[Martin Thomson [:mt:]]

I'm concerned about the metaphor.  The hailer icon implies broadcast.  I think that a bell is a much closer match, though that has the unfortunate existing use for alarms, which might be a confusing near miss.

As for whether to show two different notifications, I think that we should be combining these two permissions prompts.  See https://docs.google.com/document/d/1yYUB4nn9Hu6vPHKp_eN_kXOG47gjteEv8QIf_l18q4w/edit

Comment 18

[Mika]

Hi Matej - I suggested to the Push team that you might be able to review the copy in the FAQ and doorhanger they created. The link to the FAQ is: https://docs.google.com/document/d/16-sqk6l_D22PFYXGaEN_8kwHrrhvvoOYx0h-OBIGF-I/edit?usp=sharing

This will be surfaced to all Firefox desktop users in GA in November.  Code hasn't landed yet, but localization would happen soon after.

Comment 19

[Matej Novak [:matej]]

(In reply to Mika from comment #18)
> Hi Matej - I suggested to the Push team that you might be able to review the
> copy in the FAQ and doorhanger they created. The link to the FAQ is:
> https://docs.google.com/document/d/16-sqk6l_D22PFYXGaEN_8kwHrrhvvoOYx0h-
> OBIGF-I/edit?usp=sharing
> 
> This will be surfaced to all Firefox desktop users in GA in November.  Code
> hasn't landed yet, but localization would happen soon after.

My edits and comments are in. Thanks.

Comment 20

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

What is the progress on this? Can we land the image?

Comment 21

[Doug Turner (:dougt)]

Update please?

Comment 22

[Bill Maggs (bmaggs)]

Attachment: Bubble icon

Comment 23

[Bill Maggs (bmaggs)]

The latest designs for the Notification permission in Control Center look like a thought bubble rather than a megaphone. See attachment from current designs for Control Center slated into 42. Should we use this for a combined Push Notifications icon for the permissions dialog?

Comment 24

[Matthew N. [:MattN]]

The two permission prompts are being combined and we just want a new icon now.

Comment 25

[Edwin Wong [:edwong]]

Need icon.

Comment 26

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

Per shorlander: We're going to use the bubble with the exclamation mark.
Bryan, could you attach the correct assets here? Thanks!

Comment 27

[bbell]

Attachment: notifications.svg

new notifications icon.

Comment 28

[bbell]

Attachment: icon-notificationsTray-16.svg

Replacement for notificationsTray

Comment 29

[bbell]

Attachment: notifications-32.svg

updated notification icon.

Comment 30

[Lina Butler [:lina]]

Thanks, Bryan! I'll update the permission prompt in bug 1192458.

Comment 31

[Lina Butler [:lina]]

Landing as part of bug 1192458, to reduce merge churn.