jcorporation / myMPD

myMPD is a standalone and mobile friendly web mpd client with a tiny footprint and advanced features.
https://jcorporation.github.io/myMPD/
GNU General Public License v3.0
419 stars 65 forks source link

Group the albums by album name only #470

Closed edhelas closed 3 years ago

edhelas commented 3 years ago

myMPD version: 7.0.2

Albums that are taggued with different artists are not grouped under the same "Album".

Expected behavior

Even if an album is actually containing different artists, it should be displayed as one unique album. You could use the musicbrainz_albumid as a way to keep things grouped.

Screenshots

Capture d’écran_2021-04-15_00-13-29

edhelas commented 3 years ago

One small (unrelated) suggestion. For the #BrowseDatabaseListList element, you can use justify-content: space-around/space-between to align the albums properly and fill all the width.

jcorporation commented 3 years ago

myMPD uses Album+AlbumArtist to distinguish albums.

justify-content: this breaks the last row, if there are fewer items.

edhelas commented 3 years ago

Regarding the justify-content, indeed it's not possible to fix the last line, you have to move to a CSS grid layout.

For the other issue, I actually have an "albumartist" tag on all those albums to group them. Maybe this tag is not prioritized properly ?

Capture d’écran_2021-04-15_19-57-17

Also regarding the album list, a few more remarks:

jcorporation commented 3 years ago

Is the albumartist tag enabled in mpd and myMPD? In my environment grouping works.

If this does not help, a debug log would be good.

Please use the discussions for enhancement requests. There is a ideas category.

Issues are only used for bugs and accepted feature requests.

edhelas commented 3 years ago

Oh yes actually they were not enabled in the mpd.conf (actually it is shown in your Wiki, my bad). I had to fully clear the cache and now it's working. Thanks for the support.

I'll put my other remarks in the ideas category :)