Closed
      
        Bug 1209999
      
      
        Opened 10 years ago
          Closed 10 years ago
      
        
    
  
String for Push Notification Permission dialog
Categories
(Core :: DOM: Push Subscriptions, defect)
Tracking
()
        RESOLVED
        FIXED
        
    
  
        
            mozilla44
        
    
  
| Tracking | Status | |
|---|---|---|
| firefox44 | --- | fixed | 
People
(Reporter: wmaggs, Assigned: jaws)
References
Details
Attachments
(1 file)
| 
        
        
         2.59 KB,
          patch         
       | 
      
           MattN
 :
              
              review+
           | 
      Details | Diff | Splinter Review | 
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.
| Reporter | ||
          Comment 1•10 years ago
           
         | 
      ||
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•10 years ago
           
         | 
      ||
(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.
| Reporter | ||
          Updated•10 years ago
           
         | 
      
Flags: needinfo?(matej)
          Comment 3•10 years ago
           
         | 
      ||
(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.
Flags: needinfo?(matej)
| Reporter | ||
          Comment 4•10 years ago
           
         | 
      ||
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.
| Reporter | ||
          Comment 5•10 years ago
           
         | 
      ||
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•10 years ago
           
         | 
      ||
(In reply to Bill Maggs (bmaggs) from comment #5)
> 
> Would you like to receive notifications from this site?
Looks good!
| Reporter | ||
          Comment 7•10 years ago
           
         | 
      ||
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?
Assignee: nobody → MattN+bmo
| Assignee | ||
          Updated•10 years ago
           
         | 
      
Assignee: MattN+bmo → jaws
Status: NEW → ASSIGNED
| Assignee | ||
          Comment 8•10 years ago
           
         | 
      ||
        Attachment #8673233 -
        Flags: review?(MattN+bmo)
          Updated•10 years ago
           
         | 
      
        Attachment #8673233 -
        Flags: review?(MattN+bmo) → review+
| Assignee | ||
          Updated•10 years ago
           
         | 
      
Keywords: checkin-needed
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
          You need to log in
          before you can comment on or make changes to this bug.
        
Description
•