-
Notifications
You must be signed in to change notification settings - Fork 424
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
docs: update docs for overrideNative #77
Conversation
@@ -99,20 +104,32 @@ Drop by our slack channel (#playback) on the [Video.js slack][slack-link]. | |||
Get a copy of [videojs-http-streaming](#installation) and include it in your page along with video.js: | |||
|
|||
```html | |||
<video id=example-video width=600 height=300 class="video-js vjs-default-skin" controls> | |||
<video-js id=vid1 width=600 height=300 class="vjs-default-skin" controls> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might be worth having a section why video-js, though, maybe just a FAQ about the late-init stuff in video.js instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I'll add an explanation and also add player.src()
as an alternate setup
README.md
Outdated
### Via MSE with the [overrideNative](#overridenative) option | ||
- Chrome Android | ||
- Edge | ||
- Mac Safari |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mac Safari is not supported yet
Description
This PR is to clean up the
overrideNative
optionSpecific Changes proposed
video-js
element, how to useoverrideNative
and prefer httpsRequirements Checklist