-
Notifications
You must be signed in to change notification settings - Fork 51
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 to playlist #5423
Add to playlist #5423
Conversation
…ire form submissions to API calls
…and collapse panels
…ix playlist form updating with canvas switches
6c67939
to
e24ef67
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great! I like how you've added missing inline documentation comments for functions. I had a couple real small questions but this looks pretty much ready to go.
Co-authored-by: Chris Colvard <chris.colvard@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
Playlist dropdown with

Add new playlist
option:Add to playlist with option to create playlist items for each structure item:

Add to playlist with option to add current track/time selection with custom title and description:

Display feedback from successful request:

Display message when the user doesn't have playlists:
