-
Notifications
You must be signed in to change notification settings - Fork 182
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
Autoplay Feature from Youtube #222
Comments
The 'Autoplay' parameter in the youtube player API controls whether the video automatically starts playing when it's loaded. Since that's not applicable here, did you mean the "automatic playing of next video" feature ? Would anything be available in the API for that ? |
Yea, the automatic playing of next is what I want to implement, not sure if there is a API for that havent had a chance to look. |
Ok, interesting feature if it could be integrated. I'll also look into it |
…cking to avoid duplicate song plays
…video. History also maintained of all videos played to avoid duplicates
is this issue still open for working? |
@lazy-sunshine sure, see the work that was started in PR #258 by @avalan4e57. It hasn't been touched in months so you can finish it up based on the feedback I left there. Feel free to ping me with any questions |
is this feature still up for grabs? how is the progress going? |
@aarondchu thanks for your interest, this is still up for grabs. Reading #231 and #258 would be a good starting point. Ping me if you have any questions |
Any updates? Is this up for grabs again? |
@k----n if the issue is open, it's up for grabs |
Is this issue still open? Where are you guys at #329 ? |
@lcpunch go for it, there's been no activity on this project lately 👍 |
I implemented autoplay the next related video in this PR: #329 Not sure if it's mergeable? It was working last time I ran it in October 2019. |
@k----n @shakeelmohamed , I just tested it and it works. The only problem that I see is when the autoplay is activated it "breaks" the line. I could fix that by adjusting the container class from min-width: 450px to 470px. |
@lcpunch ok, we can make the container wider, that should be fine |
- Changes button title to follow standard - Some minor improvements - Changes container width to fit new button
@lcpunch is this one resolved? |
@ryekerjh nope, if the issue is open it's not done |
Is the issue solved? |
@Omkar0803 hi, please go ahead. It may take me a while to review the PR, however. |
Can you assign this issue to me |
Hi @shakeelmohamed , I would like to work on this issue. Can you please assign it to me? |
@PraveenRaj1833 go ahead |
Hi @shakeelmohamed, I have raised a PR for this feature. Please review it. |
I'm looking to add the "Autoplay" feature from Youtube (can have a on/off toggle) I use it all the time when streaming music on Youtube and think it would be a great feature added here.
A starting point might be here to check the API and get some sample code.
The text was updated successfully, but these errors were encountered: