Changeset 27622

Timestamp:

03/19/2014 06:59:31 PM (13 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);