Switch to HTML5 player for Vimeo

Here's a first attempt at a patch. It doesn't include frameborder, since the HTML5 conformance validator didn't like it. Also does not include things like bylines and links, which it probably should considering the default embed code Vimeo provides contains those as well.

(The patch is against contrib/emvideo/providers/vimeo.inc)

Renaming it "Universal" instead of "HTML5". Also adding options and removing a debugging statement. And finally, the attached patch is against 6.x-dev.

Make the universal player optional for now. Default to using Flash.

Thanks for your work on this Richard. I'm exciting to try to implement this on my site.

Subscribing...

This patch works like a charm. I posted my first screencast on my Drupal site last night and realized the video did not work when I went to test on my iPad even though it worked fine when on vimeo's website. I was about to file an issue here and found this one. Thank you so much! Awesome work.

Just an odd thing I noticed, I have a vimeo plus account and there is a setting that says, "Alternate versions - Make mobile versions of all my videos" When I had that checked the embedded video did not work from my website on an iPhone (still worked fine on an ipad) but when i unchecked that setting, the video proceeded to work on the iPhone so I suspect there was a conflict. I think vimeo had that setting before they had the universal player. I will continue to experiment with this.

Subscribing...

This is going to be awesome! Thanks guys for all the hard work.

At least for me, after updating to the latest module and applying the patch, when I 'embed' the code with Vimeo's new version, I simply get 'Sorry, This Video Does Not Exist'. The old embed code works fine.

Anything weird about iFrames here?

Chuckd, did you clear your cache and try toggling the universal player check box?

highrockmedia: re: #7, what did you find out about the mobile version of the player on the iPhone? I 'embedded' a video I found that had its mobile version available (City Traffic) since I don't have a Plus account. I was able to play it with the Quicktime player in my iPhone 4.

Richard, re: #7, it seems as though with the new universal player, you no longer need a plus account to have mobile versions work. The next time I upload a screen cast I'll know for sure, it might not be for a week or so, I'll post back.

Subscribing

+1 thanks and subscribing!

Gee, who knew actually toggling the universal player check box to ON would make it work? (sheepish grin)

This works great except for the fact that on the Preview display it's not working. Any reason that might be? Thanks for the great work!

I can reproduce this with the display settings in my field (admin/content/types/video/display, for example) with the Teaser set to "Preview Video". Attached is an updated patch which aims to rectify that.

That did it. Thanks so much. Great work on this. Hopefully this can get committed soon.

This has been committed to the Media: Vimeo module. Thanks, @Richard Eriksson!

The only thing I don't like about this new player is that it seems to add a 2px inset border. I had to add the following to my theme's css file to get rid of it:

#media-vimeo-1 iframe {
  border:0;
  margin:0;
  padding:0;
}