Changeset 29457 for trunk/src/wp-includes/media.php

Timestamp:

08/09/2014 08:43:25 PM (11 years ago)

Author:

wonderboymusic

Message:

Upgrade MediaElement to 2.15.0:

• It is required to set height on the parent .wp-video wrapper for video
• Fixes the play button hover problem in Chrome
• Includes my pull request: ​https://github.com/johndyer/mediaelement/pull/1273

All changes:
​https://github.com/johndyer/mediaelement/compare/2.14.2...master

See #29110.

File:

• trunk/src/wp-includes/media.php (modified) (1 diff)

trunk/src/wp-includes/media.php

@@ -1875 +1875 @@
     $html .= '</video>';
 
-    $width_rule = '';
+    $width_rule = $height_rule = '';
     if ( ! empty( $atts['width'] ) ) {
-        $width_rule = sprintf( ' width: %dpx;', $atts['width'] );
-    }
-    $output = sprintf( '<div style="%s" class="wp-video">%s</div>', $width_rule, $html );
+        $width_rule = sprintf( 'width: %dpx; ', $atts['width'] );
+    }
+    if ( ! empty( $atts['height'] ) ) {
+        $height_rule = sprintf( 'height: %dpx; ', $atts['height'] );
+    }
+    $output = sprintf( '<div style="%s%s" class="wp-video">%s</div>', $width_rule, $height_rule, $html );
 
     /**