891014 - [Music] [User story] Display an icon in the status bar while music is still playing

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Description

[Sri Kasetti]

User Story:
As a user, when I exit the music application while the music is still playing I want to see an icon on the status bar indicating that music is still being played while the home screen is still unlocked

Acceptance Criteria:
1. The user exits the music application and keeps the screen in unlocked state. He/she can find a visible icon on the status bar indicating that the music is still playing
2. If the user finds the play icon on the status bar and decides to take some action, he/she can swipe the notifications icon or swipe down from the top of the screen. This action opens up the utility tray  with details of the current song at the bottom of the utility tray
3. The utility tray should display the song playing, the album art and the music icon 
4. The user by selecting the song in the utility tray can go to the music app to take some action

Comment 1

[(inactive after 6/18) Alive Kuo [:alive]]

If I understand this bug correctly, it a simple one:
* Use mozChromeEvent: audio-channel-changed event
* When evt.detail.channel is changed into 'content': show the icon.
* When evt.detail.channel is changed into all other: hide the icon.

Comment 2

[Dominic Kuo [:dkuo]]

Update:
Working branch: https://github.com/dominickuo/gaia/tree/bug-891014

Comment 3

[Dominic Kuo [:dkuo]]

Attachment: Music Player 1.2 release spec., v0.2

Comment 4

[Dominic Kuo [:dkuo]]

Hi Eric, currently we don't a playing icon on the status bar, can you please provide the new icon for this user story? thanks.

Comment 5

[Eric Pang [:epang] UX]

Attachment: Statusbar Icons.zip

Hi Dominic,
I've added the play icon to the status bar icons at @1x and @2x (it's the last icon).  The only thing I'm worried about is that I'm not sure if I've worked off the latest version.  Is there a way we can pull the latest so I know I've added the play icon to the latest version?  Let me know if you find out it's not the latest - if you point me to the latest I can update it :)

Thanks!

Comment 6

[Dominic Kuo [:dkuo]]

Eric, thanks for the new images, and I think we also need the icons at @1.5x.

You can just pull the latest gaia master branch to see the latest images, and you probably need do the daily update because the master branch is always changing. So I will suggest you just look them on github which should be easy and fast to check the latest version.

All the latest images should be in the master(If not, that should be some bugs...):
https://github.com/mozilla-b2g/gaia/tree/master

And the image we are looking for this issue is:
https://github.com/mozilla-b2g/gaia/blob/master/apps/system/style/statusbar/images/icons.png

Comment 7

[Alex Keybl [:akeybl]]

koi+ per https://wiki.mozilla.org/B2G/Roadmap

Comment 8

[Eric Pang [:epang] UX]

Attachment: [e-mail]

Hi Dominic, I've attached the image @1.5x. Let me know if anything else is needed! Thanks :)

Comment 9

[Dominic Kuo [:dkuo]]

Hi Eric, while I was replacing the current [e-mail] to attachment 784975 [details], I found the battery icon in the second line, the first one looks slightly different, for 1.5x the border is red, but for 1x and 2x are white, not sure which one is correct and we will need these three sync, thanks.

Comment 10

[Eric Pang [:epang] UX]

Attachment: [e-mail]

Hi Dominic, I've updated the icon so it has a white outline and matches 1x and 2x.  Sorry I didn't catch that myself!  Let me know if there's anything else that's needed. Thanks!

Comment 11

[Eric Pang [:epang] UX]

Attachment: [e-mail]

Attached the wrong files.  Please use this one, thanks!

Comment 12

[Dominic Kuo [:dkuo]]

Attachment: Show play icon on the status bar when music is playing

Alive,

This patch basically implemented what you suggested in comment 1, with the new 1x, 1.5x and 2x icons images. Besides adding a test case on this new feature, I also wrote four cases that covers the mozChromeEvent changes, which were not there and looks all related to the media features, so I called it "media information" test suite, can you please review it? thanks.

Comment 13

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 795437 [details]
Show play icon on the status bar when music is playing

Thanks for add tests!

Comment 14

[Dominic Kuo [:dkuo]]

Thanks Alive!

This bug will only cover the first Acceptance Criteria in comment 0:

1. The user exits the music application and keeps the screen in unlocked state. He/she can find a visible icon on the status bar indicating that the music is still playing

The others are in bug 902974 and bug 902981.

Landed on master: e0588920dc1cf4d3cfeefc65c89d44cf3ff4f12c

Comment 15

[Dominic Kuo [:dkuo]]

Attachment: http://people.mozilla.com/~dkuo/B2G/Gaia/TestMedia/Demo_video_play_icon_in_status_bar.mp4

Attaching the link of demo video for UX/Prod to sign-off.

Comment 16

[Dominic Kuo [:dkuo]]

Sri/Rob,

Would you please review the demo of this user story in attachment 796431 [details]? thanks.

Comment 17

[Sri Kasetti]

Dominic, 

I reviewed the demo and it looks good.

Comment 18

[Wayne Chang [:wchang]]

Hi Sri,

Does this cover FM radio playing as well or should we consider extending it to?
As requested by partner in bug 908584.

Comment 19

[Sri Kasetti]

I don't think this covers the FM Radio app. Let's check with Dominic to be sure

Comment 20

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to Sri Kasetti from comment #19)
> I don't think this covers the FM Radio app. Let's check with Dominic to be
> sure

AFAIK this does. If it doesn't, it's a bug falling under audio channel service.

Comment 21

[Dominic Kuo [:dkuo]]

I took a quick test and found FM radio use the same audio channel as well(content channel), so the play icon also shows up when FM radio is playing! I am setting the dependency and closing bug 908584.

Comment 26

[Rob MacDonald [:robmac]]

Hi Dominic... 

Thanks for putting the video together. It looks good. Can you do a screen shot of what shows in the activity tray? One of the items we had for MVP from the UX perspective was that we show the album art, song title and artist in the utility tray and then link that to now playing. Assuming we can't get playback controls yet, we'd hope to see that instead.

The idea is that, if we're providing the play icon, users will need to know what it's for and be able to access it. 

Which brings up the topic of FM Radio... If we're showing a play icon in this case, then we'd apply the same logic and would like a link to the FM Radio app from within the utility tray during playback. Something like...

------------------------------
[icon]  FM Radio           >
        94.2        
------------------------------

Just try to visualize. :)

I know this wasn't anticipated but let's discuss whether this is feasible.

Rob

Comment 27

[Dominic Kuo [:dkuo]]

No problem, and the information for FM in the activity tray is a good topic, let's file a new bug and discuss in it!

Comment 28

[Intermittent Failures Robot]

5 failures in 4124 pushes (0.001 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 5

Platform and build breakdown:

• macosx1015-64-shippable-qr: 2
  • opt: 2
• macosx1100-aarch64-shippable-qr: 1
  • opt: 1
• linux1804-64-shippable-qr: 1
  • opt: 1
• macosx1015-64-qr: 1
  • opt: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=891014&startday=2024-07-22&endday=2024-07-28&tree=all