dweymouth / supersonic

A lightweight and full-featured cross-platform desktop client for self-hosted music servers
GNU General Public License v3.0
675 stars 26 forks source link

Add ability to see cover images of songs in Favorites/All Tracks/etc #276

Closed SnipeXandrej closed 1 month ago

SnipeXandrej commented 7 months ago

Show a song's cover image right next to a song's name, similarly to what Spotify does.

image

Supersonic currently does not show the cover image of each song.

image

I would mainly like to see this implemented because I find it much easier to find a song by looking for a specific cover image, rather than trying to find a song by looking at a wall of text.

Thanks!

dweymouth commented 7 months ago

Yes I agree this would be nice to have and I do plan on adding it. I'm not sure where the setting to toggle between the compact tracklist view and expanded one with images should be. Maybe in the settings dialog, and it would apply to all tracklists throughout the app. (ie if you turn it on, it would affect album pages as well even though 99% of the time, all tracks on an album have the same cover art).

Does this make sense or do you have any other ideas of how to make this configurable in the UI?

SnipeXandrej commented 7 months ago

I wonder if there should be a new settings tab named "UI"/"User Interface" or something similar (can't think of anything better right now)? Right now there is probably going to be just that one setting, but at least it's gonna be future proofed for any new features that are going to require that tab.

As for where it should be applied... I think it can be applied to all tracklists throughout the app, except when looking at an album tracklist. Enabling cover images on each track in an album is pretty unnecessary.

dweymouth commented 7 months ago

I think it's a good idea to have a new settings tab. When I get around to this I might add a checkbox for whether to enable it additionally on album pages as well, because sometimes albums are being released with different images per track, especially digital-only releases. This is quite uncommon though.

adamshand commented 7 months ago

I really like how minimalistic Supersonic is. I don't have any objection to adding album covers, but instead of adding new configuration options, is it reasonable to figure out a way to do this that will "just work" for most people?

dweymouth commented 7 months ago

I think sensible defaults, combined with burying customization in the settings UI rather than on the pages themselves is the way to go about it. I also really want to be able to maintain the minimal UI as more features are added. I really don't want to clutter the main pages but I'm a little less concerned about adding more settings tabs, within reason. (Actually, there are already a few things that can be changed only in the config file, like allowing multiple instances at once)

I think if/when this particular feature is added, the default would be to show the expanded tracklist rows with covers on everything but the album pages.