1209999 - String for Push Notification Permission dialog

Status: RESOLVED FIXED

(Core :: DOM: Push Subscriptions, defect)

Description

[Bill Maggs (bmaggs)]

We need a new string to explain the consolidated permission for receiving and showing Push Notifications for Fx 44. When the user sees this dialog, they are granting permission for a web page to send Push notifications to Firefox  and also to show Notifications on screen in Firefox.

Comment 1

[Bill Maggs (bmaggs)]

Proposed string alternatives: 

Would you like to receive Notifications from this site? 

This is accurate and covers the Push and Web Notifications included in the permission. 

Would you like to allow Notifications from this site?  

This language aligns with Control Center and Notification Management in Preferences

Comment 2

[Lina Butler [:lina]]

(In reply to Bill Maggs (bmaggs) from comment #1)
> Would you like to receive Notifications from this site? 
> 
> This is accurate and covers the Push and Web Notifications included in the
> permission.

I like "receive." As you say, it implies both local and remote notifications.

Comment 3

[Matej Novak [:matej]]

(In reply to Kit Cambridge [:kitcambridge] (PTO from 2015-10-06 to 2015-10-16) from comment #2)
> (In reply to Bill Maggs (bmaggs) from comment #1)
> > Would you like to receive Notifications from this site? 
> > 
> > This is accurate and covers the Push and Web Notifications included in the
> > permission.
> 
> I like "receive." As you say, it implies both local and remote notifications.

Copy looks good to me, just wondering why Notifications is capitalized.

Comment 4

[Bill Maggs (bmaggs)]

Does not need to be. We wanted to signal that these notifications, based on a new Push standard,  are a new type, unlike the existing local and SimplePush notifications we have had for a while. I would think after they become more popular we could remove the capital letter.

Comment 5

[Bill Maggs (bmaggs)]

My mom is an English teacher, and I hear her now telling me that despite its convenience, Notifications just isn't a proper noun. We can explain what it means in the Learn More link. So unless there are more concerns, the string is 

Would you like to receive notifications from this site?

Comment 6

[Tanvi Vyas[:tanvi]]

(In reply to Bill Maggs (bmaggs) from comment #5)
> 
> Would you like to receive notifications from this site?

Looks good!

Comment 7

[Bill Maggs (bmaggs)]

Assigning this to MattN to change this one word in the permissions dialog because of Kit being out of the office for an extended period of time. 
 From: Would you like to show notifications from this site?
 To:   Would you like to receive notifications from this site?

Comment 8

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch

Comment 9

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4bea2339e0ed

Comment 10

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

https://hg.mozilla.org/mozilla-central/rev/4bea2339e0ed