Closed
Bug 1247788
Opened 8 years ago
Closed 8 years ago
Add "Notifications" in Settings
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox48 fixed)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: antlam, Assigned: sebastian)
References
Details
Attachments
(2 files, 1 obsolete file)
We're doing a lot of things with Notifications in Fennec and right now, and the system offers a _really_ easy way to block ALL notifications. That's going to be even more appealing to users if we don't give them some level of control to manage this within our own app. I think we need to add a "Notifications" item inside Settings. WRT placement, I don't have huge opinions right now, but I think it needs to be on the first level. I'm thinking above "Advanced" since this is something that affects all users. Copy TBD, but a simple list with toggles will do I think: +-----------------------------------------+ | | | What's new update O-- | | | | | | Unread Bookmarks O-- | | | | | | Website updates O-- | | | | | | Tab queue O-- | | | | | | | | | | | | | | | | | | | | | +-----------------------------------------+
Reporter | ||
Comment 1•8 years ago
|
||
^ Items in list for illustrative purposes only
Comment 2•8 years ago
|
||
I agree we'll need something like this. I'm wondering if we should add "Tab queue" to this list. It's the only one of the notifications that is user initiated, by tapping a link in an external application. The others are opt-out and should have settings here, and maybe in the notification itself.
Comment 3•8 years ago
|
||
Yeah, tab queue feels a bit out of place. If we include tab queue, should we also include other notifications that stem from user action, like downloads? Seems like this could quickly get out of hand. I would argue that we should put an opt-out for snippets in here. Make this less about the system notification UI, more about the things Firefox could be doing to annoy you.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → s.kaspari
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•8 years ago
|
||
I landed this new settings category as part of bug 1241810: https://hg.mozilla.org/integration/fx-team/rev/d0fd7d32ee5e @antlam: I only added "content notifications" in there. I'd use this bug to add other notifications to this menu. Do you have a list of notifications you want to see there?
Flags: needinfo?(alam)
Assignee | ||
Comment 6•8 years ago
|
||
(In reply to Sebastian Kaspari (:sebastian) from comment #5) > I landed this new settings category as part of bug 1241810: > https://hg.mozilla.org/integration/fx-team/rev/d0fd7d32ee5e This is the actual patch: https://hg.mozilla.org/integration/fx-team/rev/6adb4ec5f1c8
Reporter | ||
Comment 7•8 years ago
|
||
(In reply to Sebastian Kaspari (:sebastian) from comment #5) > I landed this new settings category as part of bug 1241810: > https://hg.mozilla.org/integration/fx-team/rev/d0fd7d32ee5e > > @antlam: I only added "content notifications" in there. I'd use this bug to > add other notifications to this menu. Do you have a list of notifications > you want to see there? I think we should start with these: What's new in Firefox Discover new features and other app updates Website updates New content from supported sites you've bookmarked Later, we can look to include controls for unread bookmarks (bug 1245642), web notifications and others as well.
Flags: needinfo?(alam) → needinfo?(s.kaspari)
Assignee | ||
Comment 8•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/43513/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/43513/
Attachment #8736659 -
Flags: review?(margaret.leibovic)
Assignee | ||
Comment 9•8 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #7) > What's new in Firefox > Discover new features and other app updates I added this one (In reply to Anthony Lam (:antlam) from comment #7) > Website updates > New content from supported sites you've bookmarked I didn't change this one yet. Using bookmarks is the current heuristic but there's interest to test others like history/top sites (bug 1253290). It would be troublesome to update the string for every experiment or have multiple versions of it and update them at runtime. What we currently are using is: > Website updates > Allow notifications for supported sites Can we find a string that does not use "bookmarked"? Or do you think it's essential to communicate this for the experiment?
Flags: needinfo?(s.kaspari)
Updated•8 years ago
|
Attachment #8736659 -
Flags: review?(margaret.leibovic) → review+
Comment 10•8 years ago
|
||
Comment on attachment 8736659 [details] MozReview Request: Bug 1247788 - Add setting for "What's new?" notification. r=margaret https://reviewboard.mozilla.org/r/43513/#review40071 Nice, thanks.
Reporter | ||
Comment 11•8 years ago
|
||
(In reply to Sebastian Kaspari (:sebastian) from comment #9) > (In reply to Anthony Lam (:antlam) from comment #7) > > What's new in Firefox > > Discover new features and other app updates > > I added this one > > (In reply to Anthony Lam (:antlam) from comment #7) > > Website updates > > New content from supported sites you've bookmarked > > I didn't change this one yet. Using bookmarks is the current heuristic but > there's interest to test others like history/top sites (bug 1253290). It > would be troublesome to update the string for every experiment or have > multiple versions of it and update them at runtime. Fair point. > What we currently are using is: > > Website updates > > Allow notifications for supported sites > > Can we find a string that does not use "bookmarked"? Or do you think it's > essential to communicate this for the experiment? Let's go with this, it involves changing the other notification copy but I think it's better. > Website updates > Discover new content from supported sites > What's new in Firefox > Learn about changes and new features after an update
Flags: needinfo?(s.kaspari)
Assignee | ||
Comment 12•8 years ago
|
||
Flags: needinfo?(s.kaspari)
Attachment #8736803 -
Flags: feedback?(alam)
Assignee | ||
Comment 13•8 years ago
|
||
Comment on attachment 8736659 [details] MozReview Request: Bug 1247788 - Add setting for "What's new?" notification. r=margaret Review request updated; see interdiff: https://reviewboard.mozilla.org/r/43513/diff/1-2/
Attachment #8736659 -
Attachment description: MozReview Request: Bug 1247788 - Add setting for "What's new?" notification. r?margaret → MozReview Request: Bug 1247788 - Add setting for "What's new?" notification. r=margaret
Reporter | ||
Comment 14•8 years ago
|
||
Comment on attachment 8736803 [details]
settings_notifications.png
Let's make these switches too.
And can we change the description copy in the What's new in Fennec pref? I want to shorten it to:
Learn about new features after an update
Flags: needinfo?(s.kaspari)
Attachment #8736803 -
Flags: feedback?(alam) → feedback-
Assignee | ||
Comment 15•8 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/7dd46c214c299c5ccca96f3ee6402af1a6429c35 Bug 1247788 - Add setting for "What's new?" notification. r=margaret
Assignee | ||
Comment 16•8 years ago
|
||
Attachment #8736803 -
Attachment is obsolete: true
Flags: needinfo?(s.kaspari)
Attachment #8737125 -
Flags: feedback?(alam)
Reporter | ||
Comment 17•8 years ago
|
||
Comment on attachment 8737125 [details]
settings_notifications.png
+++++++++
Attachment #8737125 -
Flags: feedback?(alam) → feedback+
Comment 18•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7dd46c214c29
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•