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

updated to Jwplayer5.9 #32

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

updated to Jwplayer5.9 #32

wants to merge 6 commits into from

Conversation

nicolasberens
Copy link
Contributor

only updated the jwplayer file to the latest version
later i will try to use html5 playback when possible

@porkcharsui
Copy link
Member

I pushed your code to supersonic/html5 since this is definitely an ongoing issue as the changes I'm looking at for improving video streaming (issue #4) will benefit from this upgrade of the video player.

@nicolasberens
Copy link
Contributor Author

next thing i wanted to do is change the player to the html5 version when possible, is anyone working on this?

@porkcharsui
Copy link
Member

I will need HTML5 <video> for #4 to work on iOS. Right now I'll be prototyping with Chrome first then Android 4 (can use 3rd party player in the mean time).

@nicolasberens
Copy link
Contributor Author

next thing i would do is to remove the swfobject library and use the jwplayer library for loading so that the html 5 player is used. i think this would be ok and coul be merged into the html5 branch, or?

@porkcharsui
Copy link
Member

Sounds good to me. I will start my commit for #4 off of the "html5" branch I cherry picked from your pull request.

@Rovanion
Copy link
Contributor

Doesn't JWPlayer try to use HTML5 first and then falls back to flash if it has to?

@timoreimann
Copy link
Contributor

It's not as simple as that but rather depends on the combination of video format and browser.

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