Changeset 42343 for trunk/src/wp-content/themes/twentyseventeen/inc/custom-header.php

Timestamp:

11/30/2017 11:09:33 PM (7 years ago)

Author:

pento

Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:

• trunk/src/wp-content/themes/twentyseventeen/inc/custom-header.php (modified) (4 diffs)

trunk/src/wp-content/themes/twentyseventeen/inc/custom-header.php

@@ -25 +25 @@
      *     An array of custom-header support arguments.
      *
-     *     @type string $default-image          Default image of the header.
+     *     @type string $default-image          Default image of the header.
      *     @type string $default_text_color     Default color of the header text.
      *     @type int    $width                  Width in pixels of the custom header image. Default 954.
@@ -31 +31 @@
      *     @type string $wp-head-callback       Callback function used to styles the header image and text
      *                                          displayed on the blog.
-     *     @type string $flex-height            Flex support for height of header.
+     *     @type string $flex-height            Flex support for height of header.
      * }
      */
-    add_theme_support( 'custom-header', apply_filters( 'twentyseventeen_custom_header_args', array(
-        'default-image'      => get_parent_theme_file_uri( '/assets/images/header.jpg' ),
-        'width'              => 2000,
-        'height'             => 1200,
-        'flex-height'        => true,
-        'video'              => true,
-        'wp-head-callback'   => 'twentyseventeen_header_style',
-    ) ) );
+    add_theme_support(
+        'custom-header', apply_filters(
+            'twentyseventeen_custom_header_args', array(
+                'default-image'    => get_parent_theme_file_uri( '/assets/images/header.jpg' ),
+                'width'            => 2000,
+                'height'           => 1200,
+                'flex-height'      => true,
+                'video'            => true,
+                'wp-head-callback' => 'twentyseventeen_header_style',
+            )
+        )
+    );
 
-    register_default_headers( array(
-        'default-image' => array(
-            'url'           => '%s/assets/images/header.jpg',
-            'thumbnail_url' => '%s/assets/images/header.jpg',
-            'description'   => __( 'Default Header Image', 'twentyseventeen' ),
-        ),
-    ) );
+    register_default_headers(
+        array(
+            'default-image' => array(
+                'url'           => '%s/assets/images/header.jpg',
+                'thumbnail_url' => '%s/assets/images/header.jpg',
+                'description'   => __( 'Default Header Image', 'twentyseventeen' ),
+            ),
+        )
+    );
 }
 add_action( 'after_setup_theme', 'twentyseventeen_custom_header_setup' );
 
 if ( ! function_exists( 'twentyseventeen_header_style' ) ) :
-/**
- * Styles the header image and text displayed on the blog.
- *
- * @see twentyseventeen_custom_header_setup().
- */
-function twentyseventeen_header_style() {
-    $header_text_color = get_header_textcolor();
+    /**
+     * Styles the header image and text displayed on the blog.
+     *
+     * @see twentyseventeen_custom_header_setup().
+     */
+    function twentyseventeen_header_style() {
+        $header_text_color = get_header_textcolor();
 
-    // If no custom options for text are set, let's bail.
-    // get_header_textcolor() options: add_theme_support( 'custom-header' ) is default, hide text (returns 'blank') or any hex value.
-    if ( get_theme_support( 'custom-header', 'default-text-color' ) === $header_text_color ) {
-        return;
-    }
+        // If no custom options for text are set, let's bail.
+        // get_header_textcolor() options: add_theme_support( 'custom-header' ) is default, hide text (returns 'blank') or any hex value.
+        if ( get_theme_support( 'custom-header', 'default-text-color' ) === $header_text_color ) {
+            return;
+        }
 
-    // If we get this far, we have custom styles. Let's do this.
-    ?>
-    <style id="twentyseventeen-custom-header-styles" type="text/css">
-    <?php
+        // If we get this far, we have custom styles. Let's do this.
+        ?>
+        <style id="twentyseventeen-custom-header-styles" type="text/css">
+        <?php
         // Has the text been hidden?
         if ( 'blank' === $header_text_color ) :
@@ -107 +113 @@
     </style>
     <?php
-}
+    }
 endif; // End of twentyseventeen_header_style.
 
@@ -117 +123 @@
  */
 function twentyseventeen_video_controls( $settings ) {
-    $settings['l10n']['play'] = '<span class="screen-reader-text">' . __( 'Play background video', 'twentyseventeen' ) . '</span>' . twentyseventeen_get_svg( array( 'icon' => 'play' ) );
+    $settings['l10n']['play']  = '<span class="screen-reader-text">' . __( 'Play background video', 'twentyseventeen' ) . '</span>' . twentyseventeen_get_svg( array( 'icon' => 'play' ) );
     $settings['l10n']['pause'] = '<span class="screen-reader-text">' . __( 'Pause background video', 'twentyseventeen' ) . '</span>' . twentyseventeen_get_svg( array( 'icon' => 'pause' ) );
     return $settings;