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

Switch to Streamable iFrame player #237

Closed
lawlmart opened this issue Sep 15, 2017 · 9 comments
Closed

Switch to Streamable iFrame player #237

lawlmart opened this issue Sep 15, 2017 · 9 comments

Comments

@lawlmart
Copy link

lawlmart commented Sep 15, 2017

We’ve removed the files attribute from our API. In the future all videos must be played using our iframe player (similar to YouTube). The embed code is now returned by the API.

@cookpete
Copy link
Owner

@lawlmart This is quite a shame.. Is there a way to programmatically control the iframe player and receive events? Otherwise there is not much point for ReactPlayer to support Streamable videos.

@lawlmart
Copy link
Author

Yes, we support player.js.

https://github.com/embedly/player.js/

@cookpete
Copy link
Owner

cookpete commented Sep 15, 2017

Thanks @lawlmart. I've just hooked it up but annoyingly there is a bug in the Streamable iframe when calling play() after the ready event where the thumbnail overlay remains on top of the playing video. It requires a user click to remove the overlay.

See example fiddle using pretty much the exact example code from the player.js readme.

@lawlmart
Copy link
Author

Thanks for catching that, I’ll get that fixed ASAP.

@lawlmart
Copy link
Author

Fiddle looks good now.

@cookpete
Copy link
Owner

Nice work on the fix @lawlmart. Next issue is that the player seems to fail karma tests when run in Travis (see failing build). I've had to skip the tests for the time being. The tests seem fine when run locally. That are pretty straightforward tests that pass on certain events/callbacks (see example test). Any ideas why this would happen?

@lawlmart
Copy link
Author

lawlmart commented Sep 16, 2017 via email

@cookpete
Copy link
Owner

CI tests currently run in Chromium 60.0.3112 on Ubuntu.

@lawlmart
Copy link
Author

lawlmart commented Sep 16, 2017

I'd figure one of the events isn't being called judging by the timeout, but it's hard to tell which. The player is passing the playerjs tester so I'm not sure what the issue is on our end. Which URL are you testing it with?

Edit: Chromium recently disabled autoplaying without user interaction. Perhaps this is an issue?

david-hub024 pushed a commit to david-hub024/React_VideoPlayer that referenced this issue Dec 23, 2018
david-hub024 pushed a commit to david-hub024/React_VideoPlayer that referenced this issue May 23, 2020
albanqoku added a commit to albanqoku/react-player that referenced this issue Feb 24, 2021
Webmaster1116 added a commit to Webmaster1116/video-player that referenced this issue May 20, 2021
webmiraclepro added a commit to webmiraclepro/video-player that referenced this issue Sep 9, 2022
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

No branches or pull requests

2 participants