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

Support Spotify playlist folder structure #61

Open
vdbee opened this issue Jan 2, 2015 · 9 comments
Open

Support Spotify playlist folder structure #61

vdbee opened this issue Jan 2, 2015 · 9 comments

Comments

@vdbee
Copy link

vdbee commented Jan 2, 2015

First of all: thanks for all the good work, I use it a lot. I would like to propose the following enhancement as I structure my Spotify playlists using folders. Is it possible to support a foldable folder structure in the webclient? Personally I would like to have the folders unfold by default.

@kingosticks
Copy link
Member

If I remember correctly this is actually a bit tricky to do since we only get the flattened playlist names so any hierarchy has to be inferred. Unfortunately the path separator is a / character but since it's perfectly valid to have these character in your playlist name you can get in a bit of a mess. I think if we want to do this we need to have it user configurable so people can decide on what separator character they want to use (if any).

@adamcik
Copy link

adamcik commented Mar 2, 2015

We've talked about exposing playlists in mopidy in a way that allows use to keep the structure. Essentially something similar to the browse interface, possibly even reusing the browse interface just using a playlists root URI.

@kingosticks
Copy link
Member

Is that in any of the roadmaps yet?

@adamcik
Copy link

adamcik commented Mar 2, 2015

Don't remember if this made it anywhere, it was originally something that I think came up as part of the "list" support. But the idea would be to have browse(ARTIST_ROOT), browse(ALBUM_ROOT) and browse(PLAYLIST_ROOT) which would also allow for things like spotify "my artists" etc

@vdbee
Copy link
Author

vdbee commented Mar 18, 2015

Although it's not a fundamental solution, maybe an additional setting that allows the user to set the depth of the folder structure?

@touwer
Copy link

touwer commented Mar 18, 2015

Moped extracts the folders using the /
https://github.com/martijnboland/moped/blob/master/src/app/playlists/playlists.js

@kingosticks
Copy link
Member

Unfortunately the path separator is a / character but since it's perfectly valid to have these character in your playlist name you can get in a bit of a mess

@antonbil
Copy link

antonbil commented May 4, 2015

In fact I have added the code myself to do this (display folder structure).
It is very easy to do; just enhance function processGetPlaylists and add some extra css.
But it is true that some items have a / in the title. This looks somewhat peculiar, but it is perfectly alright to select such an item.
I would be very glad if this could be built in, if you want I have the code available.

@kingosticks
Copy link
Member

Thanks but I'm not sure. I believe it's on the Mopidy road map to address
this by providing a more browse-like API for playlists. So the code will
likely have to change again.
On 4 May 2015 20:38, "antonbil" notifications@github.com wrote:

In fact I have added the code myself to do this (display folder structure).
It is very easy to do; just enhance function processGetPlaylists and add
some extra css.
But it is true that some items have a / in the title. This looks somewhat
peculiar, but it is perfectly alright to select such an item.
I would be very glad if this could be built in, if you want I have the
code available.


Reply to this email directly or view it on GitHub
#61 (comment)
.

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

5 participants