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

Playlist append/prepend methods #102

Closed
wants to merge 4 commits into from
Closed

Playlist append/prepend methods #102

wants to merge 4 commits into from

Conversation

oct8cat
Copy link

@oct8cat oct8cat commented Oct 26, 2012

  • append() is the alias for add() method
  • prepend() is the method to add a media to the start of the playlist

@happyworm
Copy link
Contributor

Looks useful. Think I'll try and spend a day or two next month dedicated to the playlist add-on.

Does the animation work ok if you spam prepend many times?
It looks like it would work, but wondered if there is a .stop(true,true) missing from the animation part.

Will review more in a week or so. Thank you.

@oct8cat
Copy link
Author

oct8cat commented Oct 26, 2012

Little demo here: http://oct8cat.koding.com/jplayer/playlist/

No animation glitches, as far as I can see (at least, in Firefox)

@oracle2b
Copy link

+1

@thepag
Copy link
Contributor

thepag commented Nov 20, 2014

This is generic notification that all users issuing pull requests must sign our CLA before we can consider merging with the jPlayer project.

@oct8cat oct8cat closed this Mar 9, 2018
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.

4 participants