Changeset 1559

Timestamp:

05/09/2015 10:47:54 AM (10 years ago)

Author:

ocean90

Message:

Rosetta Theme: Don't fall back to deprecated "shots" if a site has no header image yet.

Make height and width flexible so you can optimize header images for HiDPI.

Location:

sites/trunk/global.wordpress.org/public_html/wp-content/themes/rosetta

Files:

• functions.php (modified) (1 diff)
• index.php (modified) (1 diff)

sites/trunk/global.wordpress.org/public_html/wp-content/themes/rosetta/functions.php

@@ -6 +6 @@
     add_theme_support( 'custom-header', array(
         'default-image' => false,
-        'header-text' => false,
-        'width' => 466,
-        'height' => 303,
+        'header-text'   => false,
+        'width'         => 466,
+        'height'        => 303,
+        'flex-height'   => true,
+        'flex-width'    => true,
     ) );
 

sites/trunk/global.wordpress.org/public_html/wp-content/themes/rosetta/index.php

@@ -24 +24 @@
                 <h3><?php the_title(); ?></h3>
 
-<?php
-    $hw = image_hwstring( HEADER_IMAGE_WIDTH, HEADER_IMAGE_HEIGHT );
-    if ( ! $header_image = get_header_image() ) {
-        $header_image = content_url( '/languages/shots/shot-' . $rosetta->locale . '.png' );
-        $hw = '';
-    }
-?>
-                <img class="shot" <?php echo $hw; ?>src="<?php echo esc_url( $header_image ); ?>" alt="Localized version screenshot" />
-                    <?php the_content(); ?>
+                <?php
+                if ( $header_image = get_header_image() ) {
+                    $hw = image_hwstring( HEADER_IMAGE_WIDTH, HEADER_IMAGE_HEIGHT );
+                    printf(
+                        '<img class="shot" %ssrc="%s" alt="" />',
+                        $hw,
+                        esc_url( $header_image )
+                    );
+                }
+
+                the_content();
+                ?>
 <?php endwhile; ?>
             </div>