ympd icon indicating copy to clipboard operation
ympd copied to clipboard

always display artist, album, track number

Open robotanarchy opened this issue 11 years ago • 9 comments

Hello,

when you have a huge music collection and search for one song, it is very hard to tell, from which band it is (cover songs or original?), or if it is a live/acoustic/instrumental version etc.

So could you please add the artist, album and track number to both the playlist view and the search results?

In case you fear that this will take up too much space, you could group albums in a cool way like (among lots of other players) cantata does:

2014-09-02-214957_1366x768_scrot

Thanks for creating this lightweight webinterface!

robotanarchy avatar Sep 02 '14 19:09 robotanarchy

A huge plug for this!

laclaro avatar Oct 12 '14 12:10 laclaro

I figured that you have to add these things to the C-source to be able to display this information in the web frontend as they are not yet requested by ympd. But in this case I end up with an unstable ympd which crashes on accessing the playlist if there are more than a couple of songs in the queue. Maybe the JS-object gets too large. I hope that I'll be able to dig a bit further soon.

http://s14.directupload.net/images/141110/mkzm24ux.png

laclaro avatar Nov 10 '14 21:11 laclaro

Cool, that you are working on this laclaro! :+1:

robotanarchy avatar Nov 11 '14 15:11 robotanarchy

I got it. The instability was caused by the direct use of some tag-fetching function. This left the value undefined whenever the tag was not present (->crash).

It would be favorable to be able to choose which tags to display as columns. This would require us to pull every tag field within mpd_client.c to be able to add columns flexible in JS.

For now, you have to recompile if you wish to have another column order, or more or less columns. You should be able to learn from the patch file where you would have to change things.

For browse-view in mpd.js: make sure you always have the same number of html-columns for playlist, song and dir types (by adding or removing empty ones in dir and playlist).

I created a pull request. Check it out here: https://github.com/notandy/ympd/pull/43

best regards Henning

laclaro avatar Nov 11 '14 17:11 laclaro

very nice :+1: :+1: :+1: lets see what @notandy says to this :)

robotanarchy avatar Nov 11 '14 22:11 robotanarchy

+1 from me. Thanks for considering this idea.

nebbishOne avatar Dec 01 '15 17:12 nebbishOne

@nebbishOne: Well, the last reply to this thread was more than a year ago. So I think it is not likely that anyone is working on this feature anymore. If you really want it, you could try to hack it in yourself and do a pull request :)

robotanarchy avatar Dec 05 '15 11:12 robotanarchy

@nebbishOne: Have a look at my fork, I included the proposed pull request.

SuperBFG7 avatar Dec 05 '15 12:12 SuperBFG7

:+1: This is a must-have for me. Great job folks!

Blum avatar Mar 03 '16 10:03 Blum