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

Aspect ratio problem (16:9 vs. 4:3) with the Flash fallback #26

Closed
nylnook opened this issue Oct 27, 2010 · 4 comments
Closed

Aspect ratio problem (16:9 vs. 4:3) with the Flash fallback #26

nylnook opened this issue Oct 27, 2010 · 4 comments

Comments

@nylnook
Copy link

nylnook commented Oct 27, 2010

Hi, and thanks for your work !

We start using MediaElement.js on a Wordpress podcast website with success (http://podcast.grenet.fr/), and I only encounter a small problem with
aspect ratio (16:9 vs. 4:3) if the Flash fallback is used, and cannot solve it on my side.
See for example this video (you may have to wait to see it) with Firefox :
http://podcast.grenet.fr/non-classe/ceremonie-de-signature-de-la-convention-enepsvinci/
a video with a 16:9 aspect ratio is stretched to 4:3.
I can't assure consistency because my videos come from multiples sources, so I am looking for a way to obtain black borders instead of stretching...
but I can't figure out how !

Thanks,
yagraph

@johndyer
Copy link
Collaborator

Yagraph,
I just added some code to all the player to do two things (1) it won't stretch your video, but will instead have black bars. (2) it can also dynamically resize the Flash window to the native size of the video. which would you prefer?

@nylnook
Copy link
Author

nylnook commented Oct 28, 2010

Thanks a lot johndyer, impressive reactivity !

I suppose (1) is the way to go, to assure consistency, and avoid surprising height or width for some videos. Black bars are commons on many screens these days, so I don't think it's a problem.

Do you plan to update your Wordpress plug-in with this fix ?

Cheers,
yagraph

@nylnook
Copy link
Author

nylnook commented Nov 10, 2010

Hi, I've token your updated media player from source to deploy it quicker, and it work already very well. Thanks, it's solved !

@johndyer
Copy link
Collaborator

This is now updated and it should show up in Wordpress shortly.

This issue was closed.
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