1054491 - Sound rocker not locked when in lock screen

Status: RESOLVED WONTFIX

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

Description

[Hubert Figuiere [:hub]]

The sound rocker isn't locked when I'm in the lock screen. This is bad because this mean that randomly in my pocket the phone will switch to either be completely annoying (volume up) or make me miss message (silent mode) when it is normally in "vibrate".

My Android phone doesn't do that.

This is important.

Flame, 2.0.

Comment 1

[Frank]

Attachment: patch to block volume keys on lock screen

The sound keys should only be locked if no music is playing. If music is playing when locked the user should still be able to adjust the volume. It should also be possible to adjust the volume on incoming calls without having to unlock the screen.

This patch checks if the System is locked and then further checks for the appropriate audio channel.

Since it seems that this is important user functionality unit tests should probably be created.

Comment 2

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

Comment on attachment 8515602 [details] [review]
patch to block volume keys on lock screen

Cannot r+ before UX agrees and having tests.

Comment 3

[Stephany Wilkes]

Flagging Jacqueline on lock screen.

Comment 4

[Stephany Wilkes]

Comment on attachment 8515602 [details] [review]
patch to block volume keys on lock screen

Jacqueline is already set to ni? here; also assigned the ui-review flag to her.

Comment 5

[Jacqueline Savory [:jsavory] UX]

After trying out the patch, I noticed that I wasn't able to adjust the volume while the music was paused. I think it might be better if when the music controls are appearing on the lockscreen, the user has the ability to adjust the volume whether the music is playing or paused. 

Flagging Rob as FYI as he is currently lead for lockscreen.

Comment 6

[Rob MacDonald [:robmac]]

(In reply to jsavory from comment #5)
> After trying out the patch, I noticed that I wasn't able to adjust the
> volume while the music was paused. I think it might be better if when the
> music controls are appearing on the lockscreen, the user has the ability to
> adjust the volume whether the music is playing or paused. 
> 
> Flagging Rob as FYI as he is currently lead for lockscreen.

Agreed. 

And thanks, Hubert, for identifying this issue.

Comment 7

[Sebastian Zartner [:sebo]]

(In reply to Frank from comment #1)
> The sound keys should only be locked if no music is playing. If music is
> playing when locked the user should still be able to adjust the volume. It
> should also be possible to adjust the volume on incoming calls without
> having to unlock the screen.

But also *only* if the screen is turned on. It doesn't make sense to allow to ajust the volume without seeing the set level.
Currently (build ID 20150820200530) you can change the sound volume while the screen is off.

Sebastian

Comment 9

[Jacqueline Savory [:jsavory] UX]

Comment on attachment 8515602 [details] [review]
patch to block volume keys on lock screen

Cleaning up old review flags

Comment 10

[BMO Automation]

Firefox OS is not being worked on