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

Prevent embedded Youtube videos from breaking in iOS #485

Merged
merged 1 commit into from
Jan 28, 2013

Conversation

mlms13
Copy link
Contributor

@mlms13 mlms13 commented Jan 25, 2013

When using Youtube's older embed code to add a video to my slideshow, the video plugin broke the video in iOS (tested on an iPhone and an iPad Mini). I fixed this by adding a quick check to make sure that the browser has support for at least FlashPlayer version 1 (iOS reports "0"). The plugin then only uses swfobject to embed content if Flash is supported by the browser.

I also removed the ? true : false ternary operator, because the expression should already evaluate to either true or false. Let me know if I'm missing something.

I re-minified the plugin using the YUI Compressor. Let me know if you'd prefer to have it minified some other way.

Mottie added a commit that referenced this pull request Jan 28, 2013
Prevent embedded Youtube videos from breaking in iOS
@Mottie Mottie merged commit 321d39d into CSS-Tricks:master Jan 28, 2013
@Mottie
Copy link
Contributor

Mottie commented Jan 28, 2013

Thanks, that looks like a good addition!

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

Successfully merging this pull request may close these issues.

2 participants