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

Add drop-in Playlist support #586

Open
wants to merge 3 commits into
base: itgmania-beta
Choose a base branch
from

Conversation

CrashCringle12
Copy link
Contributor

This PR is split from and dependent upon the implementation of the Sort Menu discussed here: #585.

...this also adds support for drop-in playlists in the Playlists folder.
Playlists are just txt files (exact format as favorites) placed in the Playlists folder. All available playlists will be shown under the Playlist category and will load the respective listing as preferred sorting. This can aid users who wish to distribute their favorites or recommendations, hold small tournaments, or just want separate listings for groups of songs.

Attaching the same video since it shows Playlists in use at the end.

SubSort2.mp4

@CrashCringle12
Copy link
Contributor Author

(Still in "Other" directory currently, but can/will move after discussing where it should go and other implementation stuffs)

@teejusb
Copy link
Collaborator

teejusb commented Jan 12, 2025

(Still in "Other" directory currently, but can/will move after discussing where it should go and other implementation stuffs)

Yeah! So I think we have two options.

  1. Other directory implies that the only those with machine access can really load the playlists, since otherwise they wouldn't have the perms to add files there. Maybe that is fine and we can limit the feature to that.

  2. If we don't do Other then the next place to consider would be in the local profiles. This is public friendly as one can modify their profiles via usb.

I think in both cases there should be some logic to determine what we do if all the songs aren't available on the machine. I would say we should either not display the playlist or display an empty one with a caution sign saying not all songs are found on the playlist. Another option would be display the songs that are found. I'm not sure out of these we should go with but open to hearing thoughts.

@stclairdaniel
Copy link

(Still in "Other" directory currently, but can/will move after discussing where it should go and other implementation
I think in both cases there should be some logic to determine what we do if all the songs aren't available on the machine. I would say we should either not display the playlist or display an empty one with a caution sign saying not all songs are found on the playlist. Another option would be display the songs that are found. I'm not sure out of these we should go with but open to hearing thoughts.

I think most preferable is a message saying not all songs are found and listing the ones which aren't found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants