Changeset 42217 for trunk/src/wp-content/themes/twentyseventeen/functions.php

Timestamp:

11/23/2017 04:08:42 AM (7 years ago)

Author:

pento

Message:

General: Reformat inline if () statements inside HTML tags.

This pattern occurs a handful of times across the codebase:

<div class="foo<?php if ( $bar ) { echo ' baz'; } ?>">

Unfortunately, it doesn't really play nicely with phpcbf, so all instances need to be removed in preperation for auto code formatting.

See #41057.

File:

• trunk/src/wp-content/themes/twentyseventeen/functions.php (modified) (1 diff)

trunk/src/wp-content/themes/twentyseventeen/functions.php

@@ -399 +399 @@
     require_once( get_parent_theme_file_path( '/inc/color-patterns.php' ) );
     $hue = absint( get_theme_mod( 'colorscheme_hue', 250 ) );
+
+    $customize_preview_data_hue = '';
+    if ( is_customize_preview() ) {
+        $customize_preview_data_hue = 'data-hue="' . $hue . '"';
+    }
 ?>
-    <style type="text/css" id="custom-theme-colors" <?php if ( is_customize_preview() ) { echo 'data-hue="' . $hue . '"'; } ?>>
+    <style type="text/css" id="custom-theme-colors" <?php echo $customize_preview_data_hue; ?>>
         <?php echo twentyseventeen_custom_colors_css(); ?>
     </style>