Changeset 41672 for trunk/src/wp-includes/customize/class-wp-customize-date-time-control.php

Timestamp:

10/02/2017 05:33:10 PM (7 years ago)

Author:

westonruter

Message:

Customize: Improve styling of date/time Customizer control.

• Let date legend be screen reader text when time is not included.
• Skip rendering containers for label (title) and description when not supplied in registered control.
• Fix margins and padding.

Amends [41670].
Props westonruter, afercia, sayedwp.
Fixes #42022.

File:

• trunk/src/wp-includes/customize/class-wp-customize-date-time-control.php (modified) (1 diff)

trunk/src/wp-includes/customize/class-wp-customize-date-time-control.php

@@ -113 +113 @@
         <# var idPrefix = _.uniqueId( 'el' ) + '-'; #>
 
-        <span class="customize-control-title">
-            {{ data.label }}
-        </span>
+        <# if ( data.label ) { #>
+            <span class="customize-control-title">
+                {{ data.label }}
+            </span>
+        <# } #>
         <div class="customize-control-notifications-container"></div>
-        <span class="description customize-control-description">{{ data.description }}</span>
-        <div class="date-time-fields">
+        <# if ( data.description ) { #>
+            <span class="description customize-control-description">{{ data.description }}</span>
+        <# } #>
+        <div class="date-time-fields {{ data.includeTime ? 'includes-time' : '' }}">
             <fieldset class="day-row">
-                <legend class="title-day"><?php esc_html_e( 'Date' ); ?></legend>
+                <legend class="title-day {{ ! data.includeTime ? 'screen-reader-text' : '' }}"><?php esc_html_e( 'Date' ); ?></legend>
                 <div class="day-fields clear">
                     <label for="{{ idPrefix }}date-time-month" class="screen-reader-text"><?php esc_html_e( 'Month' ); ?></label>