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)

defect

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:
Clone from brother
Attached file logcat (deleted) —
Clone from brother
Attached image pic (deleted) —
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.
Attached file It can resolve this issue (deleted) —
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)
Blocks: 1098152
Blocks: 994991
Whiteboard: ux-most-wanted-nov2014
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
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.

Attachment

General

Creator:
Created:
Updated:
Size: