Changeset 60645

Timestamp:

08/18/2025 08:25:45 PM (4 months ago)

Author:

joedolson

Message:

Accessibility: Make buttons to add site images look like buttons.

The buttons to add site icons, logos, header images, and background images were designed with a dashed border. This led to confusion with users thinking they were drop regions for image uploads, rather than buttons to trigger an upload.

Change design of upload buttons to clarify expected user interaction.

Props emaildano, afercia, joedolson, ankitkumarshah, logicrays, shailu25.
Fixes #47579.

Location:

trunk/src

Files:

• js/_enqueues/wp/customize/views.js (modified) (1 diff)
• wp-admin/css/customize-controls.css (modified) (2 diffs)
• wp-admin/css/forms.css (modified) (1 diff)
• wp-admin/options-general.php (modified) (1 diff)
• wp-includes/customize/class-wp-customize-header-image-control.php (modified) (2 diffs)
• wp-includes/customize/class-wp-customize-media-control.php (modified) (1 diff)
• wp-includes/customize/class-wp-customize-site-icon-control.php (modified) (1 diff)

trunk/src/js/_enqueues/wp/customize/views.js

@@ -38 +38 @@
         setButtons: function() {
             var elements = $('#customize-control-header_image .actions .remove');
+            var addButton = $('#customize-control-header_image .actions .new');
+
             if (this.model.get('choice')) {
                 elements.show();
+                addButton.removeClass('upload-button');
             } else {
                 elements.hide();
+                addButton.addClass('upload-button');
             }
         }

trunk/src/wp-admin/css/customize-controls.css

@@ -1440 +1440 @@
     height: auto;
     white-space: normal;
+}
+
+.customize-control .attachment-media-view .upload-button {
+    width: 100%;
+    padding: 9px 0;
+}
+
+.customize-control .attachment-media-view .upload-button.control-focus {
+    width: auto;
+    padding: 0 10px;
+}
+
+.customize-control.customize-control-header .actions .upload-button.button.new {
+    width: 100%;
+    padding: 9px 0;
 }
 
@@ -2991 +3006 @@
     }
 
+    .customize-control .attachment-media-view .upload-button {
+        padding: 5px 0;
+    }
+
     #customize-control-changeset_status .customize-inside-control-row {
         padding-top: 15px;

trunk/src/wp-admin/css/forms.css

@@ -790 +790 @@
 }
 
-.button-add-site-icon {
-    width: 100%;
-    cursor: pointer;
-    text-align: center;
-    border: 1px dashed #c3c4c7;
-    box-sizing: border-box;
-    padding: 9px 0;
-    line-height: 1.6;
-    max-width: 270px;
-    border-radius: 4px;
-    background: #f0f0f1;
-}
-
-.button-add-site-icon:focus,
-.button-add-site-icon:hover {
-    background: #fff;
-}
-
-.button-add-site-icon:focus {
-    background-color: #fff;
-    border-color: #3582c4;
-    border-style: solid;
-    box-shadow: 0 0 0 1px #3582c4;
-    outline: 2px solid transparent;
-}
-
 /*------------------------------------------------------------------------------
   15.0 - Comments Screen

trunk/src/wp-admin/options-general.php

@@ -106 +106 @@
     wp_enqueue_script( 'site-icon' );
 
-    $classes_for_upload_button = 'upload-button button-add-media button-add-site-icon';
+    $classes_for_upload_button = 'upload-button button-hero button';
     $classes_for_update_button = 'button';
     $classes_for_wrapper       = '';

trunk/src/wp-includes/customize/class-wp-customize-header-image-control.php

@@ -174 +174 @@
 
                 <# } #>
-            <# } else { #>
-
-            <div class="placeholder">
-                <?php _e( 'No image set' ); ?>
-            </div>
-
             <# } #>
         </script>
@@ -247 +241 @@
                 <?php if ( current_user_can( 'upload_files' ) ) : ?>
                 <button type="button"<?php echo $visibility; ?> class="button remove" aria-label="<?php esc_attr_e( 'Hide header image' ); ?>"><?php _e( 'Hide image' ); ?></button>
-                <button type="button" class="button new" id="header_image-button" aria-label="<?php esc_attr_e( 'Add Header Image' ); ?>"><?php _e( 'Add Image' ); ?></button>
+                <button type="button" class="button new <?php echo ! $this->get_current_image_src() ? '' : 'customize-header-image-not-selected'; ?>" id="header_image-button" aria-label="<?php esc_attr_e( 'Add Header Image' ); ?>"><?php _e( 'Add Image' ); ?></button>
                 <?php endif; ?>
             </div>

trunk/src/wp-includes/customize/class-wp-customize-media-control.php

@@ -197 +197 @@
             <div class="attachment-media-view">
                 <# if ( data.canUpload ) { #>
-                    <button type="button" class="upload-button button-add-media" {{{ describedByAttr }}}>{{ data.button_labels.select }}</button>
+                    <button type="button" class="upload-button button" {{{ describedByAttr }}}>{{ data.button_labels.select }}</button>
                 <# } #>
                 <div class="actions">

trunk/src/wp-includes/customize/class-wp-customize-site-icon-control.php

@@ -127 +127 @@
             <div class="attachment-media-view">
                 <# if ( data.canUpload ) { #>
-                    <button type="button" class="upload-button button-add-media"><?php echo $this->button_labels['site_icon']; ?></button>
+                    <button type="button" class="upload-button button"><?php echo $this->button_labels['site_icon']; ?></button>
                 <# } #>
                 <div class="actions">