Changeset 44796 for trunk/src/wp-includes/widgets/class-wp-widget-media-gallery.php

Timestamp:

03/05/2019 05:49:00 PM (7 years ago)

Author:

afercia

Message:

Accessibility: Replace media placeholder clickable divs with buttons.

<button> elements are natively interactive, supported by any assistive technology, and must be used instead of non-semantic, non-accessible <div> elements.

Also, this change aligns the Media Widgets and the Customizer site icon and site logo controls with the design pattern used in the new Block Editor for similar controls.

Props ramonopoly, welcher, afercia.
Fixes #43151.

File:

• trunk/src/wp-includes/widgets/class-wp-widget-media-gallery.php (modified) (1 diff)

trunk/src/wp-includes/widgets/class-wp-widget-media-gallery.php

@@ -229 +229 @@
                         <# } else { #>
                                 <div class="attachment-media-view">
-                                        <p class="placeholder"><?php echo esc_html( $this->l10n['no_media_selected'] ); ?></p>
+                                        <button type="button" class="placeholder button-add-media"><?php echo esc_html( $this->l10n['add_media'] ); ?></button>
                                 </div>
                         <# } #>