Changeset 59825 for trunk/src/wp-includes/class-wp-customize-nav-menus.php

Timestamp:

02/14/2025 08:32:48 PM (11 months ago)

Author:

joedolson

Message:

Customize: Accessibility: Restore heading semantics in sections.

Remove role="presentation" from headings in the customizer. These were needed to avoid confusing semantics when the headings also acted as buttons to control accordions, changed in [59924].

Change responsive CSS to use screen-reader-text styling rather than display: none so that mobile retains the headings hierarchy.

Props joedolson, hbhalodia, mikinc860, guillaumeturpin, rcreators, tirth03, dhrumilk .
Fixes #62215.

File:

• trunk/src/wp-includes/class-wp-customize-nav-menus.php (modified) (2 diffs)

trunk/src/wp-includes/class-wp-customize-nav-menus.php

@@ -1223 +1223 @@
         ?>
         <div id="<?php echo esc_attr( $id ); ?>" class="accordion-section">
-            <h4 class="accordion-section-title" role="presentation">
+            <h4 class="accordion-section-title">
                 <button type="button" class="accordion-trigger" aria-expanded="false" aria-controls="<?php echo esc_attr( $id ); ?>-content">
                     <?php echo esc_html( $available_item_type['title'] ); ?>
@@ -1258 +1258 @@
         ?>
         <div id="new-custom-menu-item" class="accordion-section">
-            <h4 class="accordion-section-title" role="presentation">
+            <h4 class="accordion-section-title">
                 <button type="button" class="accordion-trigger" aria-expanded="false" aria-controls="new-custom-menu-item-content">
                     <?php _e( 'Custom Links' ); ?>