[WIP] mostly implements display of all tracks from an artist
AbandonedPublic

Authored by mgallien on Sun, Oct 6, 3:28 PM.

Details

Reviewers
ngraham
astippich
jguidon
Group Reviewers
Elisa
Summary

add a new item in the display of all albums from an artist, a way to get all its tracks

should be improved visually speaking (a dedicated icon, a combination of the tracks cover, ...)

please provide feedback

Diff Detail

Repository
R255 Elisa
Branch
tracksFromArtist
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17364
Build 17382: arc lint + arc unit
mgallien requested review of this revision.Sun, Oct 6, 3:28 PM
mgallien created this revision.
mgallien retitled this revision from mostly implements display of all tracks from an artist to [WIP] mostly implements display of all tracks from an artist.Sun, Oct 6, 3:29 PM

No need to hurry, I had started that a long time ago.

Mostly interested on feedback about the idea.

ngraham requested changes to this revision.Sun, Oct 6, 4:11 PM

Conceptually there's now some duplication because there are two ways to view or add or play all of the tracks for all of the artist's albums: you can use either the toolbar buttons, or you can use the "All Tracks" item. It feels a bit messy IMO. I might recommend removing the hover buttons from the "All Tracks" item to address this.

Specific issues encountered while testing:

  1. The "Play Now" and "Add to Playlist" buttons in the toolbar crash Elisa when clicked and the "All Tracks" item is visible in the grid below
  2. The "Play Now" and "Add to Playlist" buttons for the new "All Tracks" item crash Elisa when clicked
  3. When in the "All Tracks" view, there needs to be a back button to leave it
  4. This item should always be sorted first, or else albums that begin with a number will appear before it
  5. The "All Tracks" view needs a lot of UI polish; don't show file paths, separate the tracks for different albums somehow, times are all displayed as 00:00, list items don't display album covers until played
This revision now requires changes to proceed.Sun, Oct 6, 4:11 PM
mgallien planned changes to this revision.Mon, Oct 7, 5:07 AM

@ngraham Thanks a lot for the feedback. I will work on it.

mgallien abandoned this revision.Tue, Oct 8, 7:26 PM

Moved to gitlab merge request