Changeset 59224 for trunk/tests/phpunit/tests/customize/nav-menus.php

Timestamp:

10/13/2024 07:07:06 PM (5 months ago)

Author:

joedolson

Message:

Administration: A11y: Fix accordion accessibility.

Change accordions in the customizer and the navigation menus to make proper usage of accordion markup patterns. This includes adding missing :focus states, using a button element to control tabbing and interaction, instead of the heading elements, and removing instructional text for screen reader users that was used to compensate for the incorrect markup pattern.

Props afercia, rishishah, kushang78, rcreators, krupajnanda, hmbashar, joedolson.
Fixes #42002.

File:

• trunk/tests/phpunit/tests/customize/nav-menus.php (modified) (3 diffs)

trunk/tests/phpunit/tests/customize/nav-menus.php

@@ -770 +770 @@
             foreach ( $post_types as $type ) {
                 $this->assertStringContainsString( 'available-menu-items-post_type-' . esc_attr( $type->name ), $template );
-                $this->assertMatchesRegularExpression( '#<h4 class="accordion-section-title".*>\s*' . esc_html( $type->labels->name ) . '#', $template );
+                $this->assertMatchesRegularExpression( '#<h4 class="accordion-section-title".*>\s*<button type="button" class="accordion-trigger" aria-expanded="false" aria-controls=".*">\s*' . esc_html( $type->labels->name ) . '#', $template );
                 $this->assertStringContainsString( 'data-type="post_type"', $template );
                 $this->assertStringContainsString( 'data-object="' . esc_attr( $type->name ) . '"', $template );
@@ -781 +781 @@
             foreach ( $taxonomies as $tax ) {
                 $this->assertStringContainsString( 'available-menu-items-taxonomy-' . esc_attr( $tax->name ), $template );
-                $this->assertMatchesRegularExpression( '#<h4 class="accordion-section-title".*>\s*' . esc_html( $tax->labels->name ) . '#', $template );
+                $this->assertMatchesRegularExpression( '#<h4 class="accordion-section-title".*>\s*<button type="button" class="accordion-trigger" aria-expanded="false" aria-controls=".*">\s*' . esc_html( $tax->labels->name ) . '#', $template );
                 $this->assertStringContainsString( 'data-type="taxonomy"', $template );
                 $this->assertStringContainsString( 'data-object="' . esc_attr( $tax->name ) . '"', $template );
@@ -789 +789 @@
 
         $this->assertStringContainsString( 'available-menu-items-custom_type', $template );
-        $this->assertMatchesRegularExpression( '#<h4 class="accordion-section-title".*>\s*Custom#', $template );
+        $this->assertMatchesRegularExpression( '#<h4 class="accordion-section-title".*>\s*<button type="button" class="accordion-trigger" aria-expanded="false" aria-controls=".*">\s*Custom#', $template );
         $this->assertStringContainsString( 'data-type="custom_type"', $template );
         $this->assertStringContainsString( 'data-object="custom_object"', $template );