wp_video_shortcode should support vimeo

[maimairel]

The latest mediaelementjs (v2.5.1) supports vimeo videos, so wp_video_shortcode should be able to handle this. Currently wp_video_shortcode only looks for youtube videos by checking the URL through regex.

This is a must-add to 4.0!

[maimairel]

Sorry, I meant v2.15.1 is the latest version of mejs :)

[helen]

It is too late for enhancements to 4.0. Perhaps 4.1 with a patch, though.

[maimairel]

Actually one line of regex matching should do the job :)

/wp-includes/media.php, line 1718

if ( ! preg_match( $yt_pattern, $src ) || ! preg_match( $vimeo_pattern, $src ) ) {

Well if not I can wait until 4.1.

Thank you!

[Tmeister]

This is a partial patch, this patch add support to Vimeo video source in the video shortcode (wp_video_shortcode).

As you can see in the image attached the video is rendered but the video player has a black bar in the bottom ( video Controls ) ​http://screencast.com/t/wlML14FUY .

What is the best way to tackle this? Adding a class to the video tag? see https://core.trac.wordpress.org/browser/trunk/src/wp-includes/media.php#L1846 and add the proper css class in the css files to hide the bar?

[wonderboymusic]

Vimeo in MEjs requires that Froogaloop is also loaded. When it is, it is breaking jQuery. I have an example patch: 29267.2.diff​

[wonderboymusic]

In 31016:

Support chromeless Vimeo via MEjs:

• Bump MediaElement script loader versions to 2.16.2 - missed in [30634], oops!
• Add Froogaloop to js/mediaelement scripts for Vimeo
• Check for Vimeo in the same locations that YouTube is checked
• Dynamically load Froogaloop script in admin when editing a TinyMCE view
• Edit MediaElement to call mejs.$.extend instead of $.extend in mejs.HtmlMediaElementShim.createPlugin(), since $ is not available via WP's jQuery (I will report this upstream):

​https://github.com/johndyer/mediaelement/blob/6f9a78e008de2de796a1797077a74a94763320f9/src/js/me-shim.js#L631

Fixes #29267.

[wonderboymusic]

Upstream: ​https://github.com/johndyer/mediaelement/pull/1394

[dimadin]

You missed to add froogaloop.min.js. I got one from ​https://github.com/vimeo/player-api/blob/master/javascript/froogaloop.min.js (with first line removed) so I'm unsure if that is the same version.

[wonderboymusic]

Thanks @dimadin - that was an svn miss on my part, I'll add it soon

[SergeyBiryukov]

[31115] missed the ticket.

[wonderboymusic]

In 31487:

Restore the changes I made in [31016] for chromeless Vimeo that were missed in [31373].

Make sure that the build file for grid doesn't load views/media-details.js, it already exists in the audio-video build.

See #29267, #28510.

[wonderboymusic]

In 31625:

When editing/viewing the details of a [video] shortcode that has Vimeo as its source, and Froogaloop has not already been lazy-loaded, check the details of the state's model for the video src, instead of the HTML element.

See #29267.