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

OGV file not recognized as OGG Video #1928

Closed
JB-Tellez opened this issue Jul 22, 2015 · 4 comments
Closed

OGV file not recognized as OGG Video #1928

JB-Tellez opened this issue Jul 22, 2015 · 4 comments

Comments

@JB-Tellez
Copy link

An Ogg video with .ogv extension not recognized. Changing extension to .ogg works. Tested on OSX 10.9.4 in Chrome and Firefox. This may be an issue with browsers themselves but thought I'd point it out as a workaround at least.

@photonstorm
Copy link
Collaborator

What's an OGV file? Is it just an OGG with a different extension, or something else?

@JB-Tellez
Copy link
Author

OGV = ogg video. At least that's what the Ogg folks are shooting for though it looks like .ogg is used for both video and audio in the wild. But many video converters output to .ogv not .ogg. Like I mentioned, this may well be an issue in the browsers themselves and there's nothing Phaser can do about it. The workaround is to just change the Ogg video extension to .ogg

Thanks for all the awesomess of Phaser, I've been a fan since FlixelPowerTools ;)

Here's a blurb from the Ogg wiki page...

** Before 2007, the .ogg filename extension was used for all files whose content used the Ogg container format. Since 2007, the Xiph.Org Foundation recommends that .ogg only be used for Ogg Vorbis audio files. The Xiph.Org Foundation decided to create a new set of file extensions and media types to describe different types of content such as .oga for audio only files, .ogv for video with or without sound (including Theora), and .ogx for multiplexed Ogg.[4] **

@photonstorm
Copy link
Collaborator

Ok thanks for the heads up and kinds words, I'll add ogv as a valid video extension to check for!

photonstorm added a commit that referenced this issue Jul 26, 2015
@photonstorm
Copy link
Collaborator

This has now been added to the dev branch - thanks for the suggestion.

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