Closed
Bug 924334
Opened 11 years ago
Closed 9 years ago
[Music] Shuffle toggle button and 'shuffle play' button should be visually distinct
Categories
(Firefox OS Graveyard :: Gaia::Music, defect, P1)
Firefox OS Graveyard
Gaia::Music
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sync-1, Unassigned)
References
Details
(Whiteboard: ux-most-wanted-nov2014)
Attachments
(3 files)
Firefox os v1.1
Mozilla build ID:20130916041201
Created an attachment (id=528244)
pic
DEFECT DESCRIPTION:
User experience is not very well when turn off shuffle button
REPRODUCING PROCEDURES:
1、launch music player->go to Artists view and choose an artist whose has several songs(eg pic1)
2、Choose to play all songs->turn off shuffle button on playscreen (eg pic2)
3、back to the previous screen,you will see the shuffle button stlii highlight(eg pic3)->KO
EXPECTED BEHAVIOUR:
the shuffle button should better turn grey
ASSOCIATE SPECIFICATION:
TEST PLAN REFERENCE:
TOOLS AND PLATFORMS USED:
USER IMPAC
moderate
REPRODUCING RATE:
3/3
For FT PR, Please list reference mobile's behavior:
I try to resolve this issue by add code in handleEvent function on music.js file.
The code as follow:
switch (target.id) {
case 'title-back':
if (pendingPick) {
if (ModeManager.currentMode === MODE_PICKER) {
pendingPick.postError('pick cancelled');
return;
}
cleanupPick();
}
ModeManager.pop();
+ asyncStorage.getItem(SETTINGS_OPTION_KEY, function(settings) {
+ playerSettings = settings;
+ var shuffleOption = (playerSettings && playerSettings.shuffle) ?
+ playerSettings.shuffle : false;
+ SubListView.shuffleButton.disabled = !shuffleOption;
+ });
break;
Created an attachment (id=535528)
It can resolve this issue
It can resolve this issue.
Comment 8•10 years ago
|
||
I think this comes from a misunderstanding of what the shuffle button means in the album/artist view. It's not a toggle button like in the player; it's a button that means "start playing this playlist in shuffled mode", so there'd never be a time where it makes sense to gray this out. Jacqueline: do you have any ideas here?
Flags: needinfo?(jsavory)
(In reply to Jim Porter (:squib) from comment #8)
> I think this comes from a misunderstanding of what the shuffle button means
> in the album/artist view. It's not a toggle button like in the player; it's
> a button that means "start playing this playlist in shuffled mode", so
> there'd never be a time where it makes sense to gray this out. Jacqueline:
> do you have any ideas here?
[Filling in for JS this week, as she's on PTO.]
Yes, to Jim's comment 8. In this case, the shuffle button is not a toggle.
In the Album view, there are 2 buttons - both are "PLAY" options. The traditional triangle PLAY button, plays the album in it's original song order. The second option is the SHUFFLE PLAY button, which plays the album in a random shuffle.
The user can turn off shuffle option (as a toggle) once the Album has begun to play (as in the original attached JPG). If user returns to Album view (taps '<' back button), both options: 1) Play in order and 2) Shuffle Play are (visibly) available to the user.
[ This may be revisited in 2.2 ]
Flags: needinfo?(jsavory)
Comment 10•10 years ago
|
||
Retitling to reflect what I assume this bug is now tracking.
Summary: [Buri][Music]User experience is not very well when turn off shuffle button → [Music] Shuffle toggle button and 'shuffle play' button should be visually distinct
Comment 11•9 years ago
|
||
We eliminated the "shuffle play" button in NGA. Resolving.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•