1241596 - Improve tab queue notification by using Inbox style

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: tabqueue-inbox-notification-style.png

I'm often baffled by how little information we show in our tab queue notification. I think a quick-win could be to use the "inbox style" for the notification and show some of the URLs queued. See the attached screenshot.

Comment 1

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

I like it! Ship it! This is something we wanted to do originally but I think we had technical difficulties. 

I think this also qualifies as the "Richer" notifications in bug 1172272.. so we might want to rename that one to be about the Actions on the notification.

Comment 2

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: MozReview Request: Bug 1241596 - Use inbox style for tab queue notification. r?

Review commit: https://reviewboard.mozilla.org/r/31817/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/31817/

Comment 3

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Comment on attachment 8710620 [details]
tabqueue-inbox-notification-style.png

YES

Comment 4

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: accent_colors.png

Antlam: Quick question, what do you think about adding an accent color? It's just a one-liner in code.

Comment 5

[Sebastian Kaspari (:sebastian; :pocmo)]

(In reply to Sebastian Kaspari (:sebastian) from comment #4)
> Created attachment 8710946 [details]
> accent_colors.png

(Wow, the colors look different on all my desktop screens)

To clarify: This color is only used on Android 5.0+ device. It's ignored on previous versions that didn't support accent colors in notifications.

Comment 6

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Comment on attachment 8710946 [details]
accent_colors.png

+ for fennec orange!

Comment 7

[Andrzej Hunt :ahunt]

Comment on attachment 8710629 [details]
MozReview Request: Bug 1241596 - Use inbox style for tab queue notification. r?

https://reviewboard.mozilla.org/r/31817/#review28705

Looks good to me, and yay for better tabqueue notifications!

Comment 8

[Sebastian Kaspari (:sebastian; :pocmo)]

https://hg.mozilla.org/integration/fx-team/rev/c77c13e5730fa2230ba19bb81a4dca8a10f0a53e
Bug 1241596 - Use inbox style for tab queue notification. r=ahunt

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/c77c13e5730f

Comment 10

[Teodora Vermesan (:TeoVermesan)]

Attachment: Screenshot from 2016-01-25 14:21:24.png

Verified as fixed using:
Device: Nexus 6 (Android 6.0)
Build: Firefox for Android 46.0a1 (2015-01-24)

Comment 11

[Liz Henry (:lizzard) (relman/hg->git project)]

Release Note Request (optional, but appreciated)
[Why is this notable]: Nice new feature visible to users.
[Suggested wording]: Tab queue notifications show more information
[Links (documentation, blog post, etc)]:

Please suggest improvements to the wording if you think of something more exciting! Thanks.

Comment 12

[Liz Henry (:lizzard) (relman/hg->git project)]

Noted for aurora as "Tab Queue notifications show as URL list"

Comment 13

[Liz Henry (:lizzard) (relman/hg->git project)]

How about "Tab Queue notification bar now lists multiple URLs opened in the background" with a link to the SUMO page that explains Tab Queue?  Is there a better source to link to, or anywhere that we explain what this feature does for users? 

Anthony, do you have a suggestion here for the wording?

BTW, I love this feature and use it every day!

Comment 14

[Sebastian Kaspari (:sebastian; :pocmo)]

Oh, yeah, this is something for the release notes! I think we only calls this "tab queue(s)" in Engineering. The setting is called "Open multiple links" - which again is not the best name for the feature. We have been discussing this in bug 1220932. Maybe we can say something like "Improved notification for tabs opened in the background" or like "Notifications for tabs opened in the background now list the URLs".

Comment 15

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #13)
> How about "Tab Queue notification bar now lists multiple URLs opened in the
> background" with a link to the SUMO page that explains Tab Queue?  Is there
> a better source to link to, or anywhere that we explain what this feature
> does for users? 
> 
> Anthony, do you have a suggestion here for the wording?

I like Sebastian's suggestion of "Improved notification for tabs opened in background". But I feel like that is more fitted for when we land bug 1172272.

Perhaps "Notification for links opened in the background now include URLs"? That seems to read a bit better also.

I also have a blog post that describes this feature but also how it came to be designed and developed: https://medium.com/@antlam/open-multiple-links-1ce475c47de3#.xismso2pw 

> BTW, I love this feature and use it every day!

Thanks! Glad you like it :lizzard! :)

Comment 16

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

^