-
Notifications
You must be signed in to change notification settings - Fork 22.4k
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
fix(html): remove ogv file from example #34514
Conversation
Preview URLs (comment last updated: 2024-07-04 06:52:26) |
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.
@OnkarRuikar The change you have made is perfectly fine, but you also need to update the corresponding description below the embedded example.
There are also some other mentions of OGG that you might want to review and update (for example in the "Server support for video" section).
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.
@OnkarRuikar looking good! A couple more minor suggestions for you to look at, but I've decided to approve this
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Awesome, thanks again @OnkarRuikar. Merging! |
The demo fails on Chromium-based browsers as they only play the audio, not the video. Firefox doesn't play ogv at all.
The demo still remains relevant after removing the ogv source, because the next inline source avi isn't supported in any browser and they skip to mp4.