NewPipe icon indicating copy to clipboard operation
NewPipe copied to clipboard

Sort local playlists in groups / folders

Open justanidea opened this issue 6 years ago • 32 comments

It could be cool to sort our local playlist, because when you have a lot of it its kinda messy. If someone would like to do that, I would advise him/her not to do folders, but things more like the animation used for "import export" in the sub page for each new category created by the user.

justanidea avatar Apr 14 '18 10:04 justanidea

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Dec 01 '18 15:12 stale[bot]

Related (with discussion): https://github.com/TeamNewPipe/NewPipe/issues/1982#issuecomment-528029200

Stypox avatar Sep 04 '19 18:09 Stypox

@Stypox https://github.com/TeamNewPipe/NewPipe/issues/1313#issuecomment-528029306... although I'm a fan of tags...I agree with you, folders make more sense here and alot easier to use too.

This would help organize all my playlist.. a special folder for all my music 🎶 playlists... I can even have it divided by artist, now that's cool 😎

MD77MD avatar Aug 27 '20 23:08 MD77MD

Damn, the me of 2 years ago had such a good idea :o And @MD77MD is on the same direction as me, i was thinking in sorting my musics too when writing this post (i usually get 1000 more musics each year...)

justanidea avatar Aug 28 '20 17:08 justanidea

this is the kind of small features which reviels how stupid and manipulative YouTube is -aka Google- and once someone tires them, can never go back to the YouTube app

MD77MD avatar Aug 28 '20 23:08 MD77MD

@Stypox @MD77MD is someone working on this?

i want to try to implement this

cntrl-s avatar Jan 15 '21 09:01 cntrl-s

@cntrl-s No, nobody is working on this at the moment, so feel free to pick it up! Thank you :-D Before taking ui / usability design decisions I suggest you to get some feedback here, so that you won't have to make more changes later ;-)

Stypox avatar Jan 15 '21 12:01 Stypox

@Stypox

Before taking ui / usability design decisions I suggest you to get some feedback here, so that you won't have to make more changes later ;-)

i will, thank you :)

cntrl-s avatar Jan 15 '21 14:01 cntrl-s

@cntrl-s any update?

i don't know coding but I can help with testing and feedback.

MD77MD avatar Jan 25 '21 19:01 MD77MD

@MD77MD i am new to android development so this will take a while

i will let you know once i have something for you to test :P

cntrl-s avatar Jan 26 '21 07:01 cntrl-s

@MD77MD @Stypox how do you think we should implement playlist group view

i am thinking horizontally scrollable (like feed group) under playlist tab with thumbnail of first playlist under the group as thumbnail for the group

cntrl-s avatar Jan 26 '21 08:01 cntrl-s

@cntrl-s I would do it like the channel groups/subscription groups. So the current playlist view gets expanded by a header to manage the groups. When clicking on one of those groups it opens the old playlist overview with all the playlists in that group. From there you can open a single playlist to see the single videos.

It would allow to reuse/copy stuff, depending if you want to extract out common components or just copy over what you need. Restriction would be that you can only have 1 layer and thus can't have nested groups, like a traditional file system. But i dare say that multilayer approach is a lot harder and not necessary.

XiangRongLin avatar Jan 26 '21 09:01 XiangRongLin

But i dare say that multilayer approach is a lot harder and not necessary.

@XiangRongLin i agree with this, i was thinking we could have simple groups like subscription groups

cntrl-s avatar Jan 26 '21 11:01 cntrl-s

JSON / YAML import / export of playlists feature would also be nice.

And maybe an archiving feature per playlist?

Guess I'll describe these ideas in separate issues, but they should be considered in possible playlists tab redesign.

tsilvs avatar Jan 26 '21 20:01 tsilvs

i agree with this, i was thinking we could have simple groups like subscription groups

@cntrl-s I also agree, but I'd use the group's thumbnail as a background image for the group item. Also, I'd make the playlist group items have: a height that is something in between 1.5x and 2x the height of subscription group items; a width/height ratio of 16:9.

Guess I'll describe these ideas in separate issues,

@JeffRockatansky yeah, those are different and unrelated proposals

Stypox avatar Jan 27 '21 13:01 Stypox

@cntrl-s I also agree, but I'd use the group's thumbnail as a background image for the group item.

@Stypox that's what i said in a reply above we could use the thumbnail of the first playlist under the group as thumbnail for the group or we could combine different playlists thumbnail to make a new one (vlc android does this for folder with multiple videos)

cntrl-s avatar Jan 27 '21 14:01 cntrl-s

@cntrl-s if this does not require storing too much cached data on the user's phone then ok

Stypox avatar Jan 27 '21 14:01 Stypox

@Stypox ok i will start working on this, thank you

cntrl-s avatar Jan 27 '21 14:01 cntrl-s

@cntrl-s thanks to you! :-D

Stypox avatar Jan 27 '21 14:01 Stypox

@JeffRockatansky @Stypox @MD77MD @XiangRongLin how do you think the playlist selection view should be when selecting playlists for a group

should every playlist have a radio button or have a ':white_check_mark:' on top of selected playlists or something else

cntrl-s avatar Feb 20 '21 13:02 cntrl-s

I personally don't care too much about it. Try to make it simple and close to the existing channel groups.

So a list where if one entry is selected the thumbnail is covered my a ✔,would be the simplest that i can think of right now

XiangRongLin avatar Feb 21 '21 09:02 XiangRongLin

@cntrl-s a tick over a playlist is good. Radio will take too much additional space. You could even dim selected list elements in addition to a check mark.

Maybe you could also try implementing paths while naming a playlist? E.g. when you type Parent/Child or Parent.Child (or with any other custom delimiter) then playlist Child is moved into playlist Parent, so the tree could look like this:

Parent
├ Child
│ └ [videos]
â”” [videos]

Parsing existing names would also be very much appreciated, especially because my playlists all look like _sw.os.linux.arch, _sci.phys.quant etc, and I have over 100 playlists already)))

In case of parsing a path, you should also consider and process certain delimiter edge cases:

  • leading, e.g. /Parent/Child
  • trailing, e.g. Parent/Child/
  • neighboring, e.g. Parent//Child

tsilvs avatar Feb 24 '21 08:02 tsilvs

@JeffRockatansky Im going to give this a hard NO, because this is complex enough as is.

Please open a seperate issue if you want the feature.

XiangRongLin avatar Feb 24 '21 16:02 XiangRongLin

@XiangRongLin ok, I get it.

What about the structure of folders itself?

Should a folder contain:

  • both videos and sublists or
  • only sublists?

tsilvs avatar Feb 24 '21 17:02 tsilvs

https://github.com/TeamNewPipe/NewPipe/issues/1313#issuecomment-767414178

XiangRongLin avatar Feb 24 '21 17:02 XiangRongLin

@XiangRongLin could you explain in detail why multilayer is harder? And why do you think it's not necessary?

tsilvs avatar Feb 24 '21 17:02 tsilvs

@XiangRongLin could you explain in detail why multilayer is harder? And why do you think it's not necessary?

Because its multiple layers, meaning more stuff to do. More changes need to be made to the data model, the logic and the representation. More needs to be tested and more code needs to be reviewed. Overall its more work, which can be split up into multiple feature request and multiple PR

XiangRongLin avatar Feb 24 '21 18:02 XiangRongLin

@JeffRockatansky i will implement it with dim and a tick

cntrl-s avatar Feb 25 '21 12:02 cntrl-s

Because its multiple layers, meaning more stuff to do. More changes need to be made to the data model, the logic and the representation. More needs to be tested and more code needs to be reviewed. Overall its more work, which can be split up into multiple feature request and multiple PR

also we have to keep track of both local and remote playlists within the folder structure which would be very complex to design database for and this would be even harder as sqlite doesn't support constraints on the database table, we probably would have to keep track of local and remote playlists separately

cntrl-s avatar Feb 25 '21 12:02 cntrl-s

I'm still hope for this feature. I have to much playlists. I hope to filter my playlists by genre (music playlists) and language.

Sam4000der3 avatar Mar 20 '22 20:03 Sam4000der3