Parul Mathur Reporter
	Description • 12 years ago

Test Environment:
Device: unagi phone
Build Identifier: 20130104070203
Update channel: nightly
Git commit info: 2013-01-04 20:20:08

Steps to reproduce:
1. Copy several music files onto the phone SD Card which has ID3 tags for Title, Artist, Album and Cover Art.
2. Launch the Music App and play any song.
3. On the Player page, tap on the album art.

Actual results:
1. Tapping on the album art brings into view the top and bottom overlay which have the artist name, album name, song rating, shuffle and loop buttons.

Expected results:
Tapping on the album art should navigate to the Album content without the bottom tabs, as per the spec.
https://www.dropbox.com/s/v1vuno1688gmy4q/B2g-music-v4.pdf (pages 8 and 7)

See screenshot attached.

	Dominic Kuo [:dkuo]
	Comment 1 • 12 years ago

Should be related to bug 831265

	Mihai Cirlanaru [:mihai][:mcirlanaru] Assignee
	Comment 2 • 12 years ago

Display the album content when tapping the album art from the music player.

	Dominic Kuo [:dkuo]
	Comment 3 • 11 years ago

Comment on attachment 754102 [details]
Pull Request #9998 - Show album content

Mihai,

I think this issue needs more discussion with UX/Visual because the original spec is out-of-date and some of the behaviors were changed while the Music bugs are being fixed, so I think probably UX/Visual expects some other behaviors for now, besides, there are newly added/removed behaviors on the back button and the album cover in your patch(please github comments), which might confuse users so I would like to hold this patch and ask for UX/Visual inputs.

	Mihai Cirlanaru [:mihai][:mcirlanaru] Assignee
	Comment 4 • 11 years ago

(In reply to Dominic Kuo [:dkuo] from comment #3)
> Comment on attachment 754102 [details]
> Pull Request #9998 - Show album content
> 
> Mihai,
> 
> I think this issue needs more discussion with UX/Visual because the original
> spec is out-of-date and some of the behaviors were changed while the Music
> bugs are being fixed, so I think probably UX/Visual expects some other
> behaviors for now, besides, there are newly added/removed behaviors on the
> back button and the album cover in your patch(please github comments), which
> might confuse users so I would like to hold this patch and ask for UX/Visual
> inputs.

Thanks for checking the patch, Dominic. I will wait for the UX team to give their input on this before revising the patch.

	Stephany Wilkes
	Comment 5 • 11 years ago

Assigning to Rob for UX feedback for possible patch revision.

	Rob MacDonald [:robmac]
	Comment 6 • 11 years ago

Hi everyone...

Although, technically, this bug is valid based on the spec, in this case the spec is unfortunately out of date. Taking the users to the artwork makes it difficult to access the repeat, rating and shuffle features. So Dominic is correct in his assumption that we should hold off on this until we have a chance to look at 1.2 features. 

Thanks for raising this discrepancy and I apologize for any confusion as a result of the outdated spec. Feel free to ping me if you have any questions or concerns but otherwise this bug can be closed.

- Rob

	Mihai Cirlanaru [:mihai][:mcirlanaru] Assignee
	Comment 7 • 11 years ago

Hi Rob,

Thanks for the details, I think we should still keep this bug to track the feature as there is no way to go to the album song list from the player screen.

Regarding the repeat, rating and shuffle features, they can still be accessed in the 5 or so seconds when the song starts playing, while afterwards they can be integrated in the album view (shuffle is already there). This can be included in bug 862706, which already tracks UI/UX for the album view when a song is playing.

Let me know what you think about this.

	Stephany Wilkes
	Comment 8 • 11 years ago

Closing this as invalid, as the specs are out of date. Subsequent design work, and future design work for 1.2, will supersede this.