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

Mobile is not supported? #2

Closed
xwlcn opened this issue Jan 7, 2017 · 6 comments
Closed

Mobile is not supported? #2

xwlcn opened this issue Jan 7, 2017 · 6 comments

Comments

@xwlcn
Copy link

xwlcn commented Jan 7, 2017

Mobile is not supported ?

@weizhenye
Copy link
Owner

Basic features should works fine. Some advanced features like \clip may not work. Not tested with much mobile browsers, but Chrome for mobile is just consistent with Chrome.

@xwlcn
Copy link
Author

xwlcn commented Jan 8, 2017

I use mobile UC browser, it doesn't work with fullscreen.

@weizhenye
Copy link
Owner

For most browsers, click fullscreen button in <video> element will only maximize video itself, the <div> to show subtitles is not fullscreened.

Generally native controls may not used in production, if you are using some library like video.js, its fullscreen button will maximize the wrapper <div>, which including <video>, the custom controls and some other <div>s. Subtitles can also be put in the wrapper <div>, then being fullscreened together.

@xwlcn
Copy link
Author

xwlcn commented Jan 8, 2017

yes, I used plyr.js, it aways auto resize the video's size when the html page loaded, and I called ass.resize() when the video enter fullscreen and exit fullscreen, it works well on PC, but mobile doesn't work. you can see this page: http://bd-dy.com/play/8909-0.htm on android mobile. Can you write an example with plyr.js?

@weizhenye
Copy link
Owner

I did some simple tests with UC browser and found that when clicking the fullscreen button, it seems OK, but once played the video, the controls of plyr.js was replaced by UC native controls.

That means plyr.js itself doesn't support UC browser, exactly, UC browser doesn't allow third library to control video when fullscreen.

So, maybe there are no way to support UC browser.

@xwlcn
Copy link
Author

xwlcn commented Jan 9, 2017

Thanks for your anwser.

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