Changeset 27622

Timestamp:

03/19/2014 06:59:31 PM (11 years ago)

Author:

wonderboymusic

Message:

On attachment pages for audio and video, display players. Currently, only a link is displayed. Add minimal CSS rules for compatibility with 2011, 2012, and 2013 themes.

In prepend_attachment, add logic to support attachment types that are not image.

In get_post_class(), don't add the has-post-thumbnail class for attachment pages.

Fixes #27243.

Location:

trunk/src

Files:

• wp-content/themes/twentyeleven/style.css (modified) (1 diff)
• wp-content/themes/twentythirteen/style.css (modified) (1 diff)
• wp-content/themes/twentytwelve/style.css (modified) (1 diff)
• wp-includes/post-template.php (modified) (2 diffs)

trunk/src/wp-content/themes/twentyeleven/style.css

@@ -1707 +1707 @@
 }
 
+/* =Media
+-------------------------------------------------------------- */
+
+audio,
+video {
+    display: inline-block;
+    max-width: 100%;
+}
+
+.attachment .entry-content .mejs-container {
+    margin-bottom: 24px;
+}
 
 /* =Navigation

trunk/src/wp-content/themes/twentythirteen/style.css

@@ -1838 +1838 @@
 }
 
+.attachment .entry-content .mejs-audio {
+    max-width: 400px;
+    margin: 0 auto;
+}
+
+.attachment .entry-content .wp-video {
+    margin: 0 auto;
+}
+
+.attachment .entry-content .mejs-container {
+    margin-bottom: 24px;
+}
 
 /**

trunk/src/wp-content/themes/twentytwelve/style.css

@@ -1008 +1008 @@
 }
 
+/* =Single audio/video attachment view
+-------------------------------------------------------------- */
+
+.attachment .entry-content .mejs-audio {
+    max-width: 400px;
+}
+
+.attachment .entry-content .mejs-container {
+    margin-bottom: 24px;
+}
+
 
 /* =Single image attachment view

trunk/src/wp-includes/post-template.php

@@ -348 +348 @@
         $classes[] = 'post-password-required';
     // Post thumbnails
-    } elseif ( current_theme_supports( 'post-thumbnails' ) && has_post_thumbnail( $post->ID ) ) {
+    } elseif ( ! is_attachment( $post ) && current_theme_supports( 'post-thumbnails' ) && has_post_thumbnail( $post->ID ) ) {
         $classes[] = 'has-post-thumbnail';
     }
@@ -1242 +1242 @@
         return $content;
 
-    $p = '<p class="attachment">';
-    // show the medium sized image representation of the attachment if available, and link to the raw file
-    $p .= wp_get_attachment_link(0, 'medium', false);
-    $p .= '</p>';
+    if ( wp_attachment_is_image() ):
+        $p = '<p class="attachment">';
+        // show the medium sized image representation of the attachment if available, and link to the raw file
+        $p .= wp_get_attachment_link(0, 'medium', false);
+        $p .= '</p>';
+    elseif ( 0 === strpos( $post->post_mime_type, 'video' ) ):
+        $meta = wp_get_attachment_metadata( get_the_ID() );
+        $atts = array( 'src' => wp_get_attachment_url() );
+        if ( ! empty( $meta['width'] ) && ! empty( $meta['height'] ) ) {
+            $atts['width'] = (int) $meta['width'];
+            $atts['height'] = (int) $meta['height'];
+        }
+        $p = wp_video_shortcode( $atts );
+    elseif ( 0 === strpos( $post->post_mime_type, 'audio' ) ):
+        $p = wp_audio_shortcode( array( 'src' => wp_get_attachment_url() ) );
+    endif;
+
     $p = apply_filters('prepend_attachment', $p);