Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement request: Add sort options and album thumbnail to Compositions browse #923

Open
audiomuze opened this issue Oct 22, 2023 · 4 comments

Comments

@audiomuze
Copy link

Presently Compositions are listed without any sort options and in a manner that makes it difficult to assimilate, listing track artist, composer, track number & track title in that sequence, and seemingly the output of a SQL query without an ORDER By clause.

image

@michaelherger, regarding the Compositions listing, any chance you could:

  • Include the album name in the query
  • Change the presentation sequence of fields to: title, track artist, album, track, composer
  • Include Sort options to sort by:
  1. title, track artist, year
  2. track artist, title, year
  3. year, title, track artist
  4. year, track artist, title
  • Provide a way for a front-end like Material to display an album cover thumbnail
  • Remove the artist in question from the Composer entry if they're the only Composer associated with the track but include additional composers if they are associated with the track. [if this one adds too much overhead, then just list them all as you're currently doing]

@CDrummond, hopefully you're planning on adding the Composition browse to Material?

@CDrummond
Copy link
Contributor

CDrummond commented Oct 22, 2023

@audiomuze I'm not adding a separate view. But "Compositions" will be listed as a separate section in the Albums/Releases list - where the artist_roles has "composer" but not "albumartist", "artist", or "trackartist".

All this is already implemented in the 'dev' branch of Material, so feel free to give it a try.

@michaelherger
Copy link
Member

I'm a little confused: didn't you submit this request already a day or two ago? I can't find that old one any more.

@audiomuze
Copy link
Author

audiomuze commented Oct 22, 2023

I'm a little confused: didn't you submit this request already a day or two ago? I can't find that old one any more.

I did, here. You asked me to submit it as an independent feature request, hence my setting it out separately, and hopefully a little more clearly.

@audiomuze
Copy link
Author

Thinking about this I’m not sure it’s needed in LMS per se, it’s likely better implemented in Material skin and if I’m not mistaken all the necessary datapoints are already available to Material.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants