Closed Bug 924334 Opened 7 years ago Closed 4 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
Clone from brother
Attached image pic
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.
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: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.