Changeset 27328 for trunk/src/wp-admin/includes/media.php

Timestamp:

02/28/2014 09:27:40 PM (11 years ago)

Author:

wonderboymusic

Message:

• Videos should always render at the same aspect ratio.
• Don't force a pseudo-mime-type for .m4v files
• Uniformly adapt to $content_width when setting video dimensions on the front end
• Add the height attribute to the initial <video> in the video playlist JS template
• Add some defensive/responsive CSS for a/v on the Edit Media page

See #27243.

File:

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

trunk/src/wp-admin/includes/media.php

@@ -2641 +2641 @@
 
         $meta = wp_get_attachment_metadata( $attachment_id );
-        $w = ! empty( $meta['width'] ) ? min( $meta['width'], 600 ) : 0;
-        $h = 0;
-        if ( ! empty( $meta['height'] ) )
-            $h = $meta['height'];
-        if ( $h && $w < $meta['width'] )
+        $w = ! empty( $meta['width'] ) ? min( $meta['width'], 640 ) : 0;
+        $h = ! empty( $meta['height'] ) ? $meta['height'] : 0;
+        if ( $h && $w < $meta['width'] ) {
             $h = round( ( $meta['height'] * $w ) / $meta['width'] );
+        }
 
         $attr = array( 'src' => $att_url );
-
-        if ( ! empty( $meta['width' ] ) )
+        if ( ! empty( $w ) && ! empty( $h ) ) {
             $attr['width'] = $w;
-
-        if ( ! empty( $meta['height'] ) )
             $attr['height'] = $h;
+        }
 
         echo wp_video_shortcode( $attr );