-
Notifications
You must be signed in to change notification settings - Fork 273
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
abandon yt js api #192
Comments
Will have to try that out. Really appreciate you filing! |
the id for playlist is ofc the video id |
I think the question before deciding to abandon (past ensuring this works as expected for the use cases) is, do we not want to expose the API? It is quite powerful for developers to control the player. Is there a way for them to attach the JS API on top of the embed if we remove it? Currently we don't expose it against the video, sure. But it has come up that some do want that made public. If we want to try and "trim everything down", then maybe loading the YT API and exposing the instance against the video is something we could let developers opt-in to. |
I think the main goal of this lib is to make as light of an yt embed as possible, the js api isn't always included anyways, only on some specific browsers, so it's not like it's a compatibility change |
and again, all this stuff can be exposed via this api spoof I showed |
youtube's JS API for the most part isn't needed, because I think the autoplay bug can be fixed by appending
&playlist={id}
, however I don't have a Mac to verify this, instead the JS API can be "spoofed":this saves loading their 200kb or so API
The text was updated successfully, but these errors were encountered: