Changeset 46778

Timestamp:

11/25/2019 11:18:04 AM (5 years ago)

Author:

SergeyBiryukov

Message:

Accessibility: Media: Add an aria-pressed attribute to active button within a button group.

The aria-pressed attribute communicates semantically the "active" state of buttons that otherwise only look "active".

Props afercia.
Merges [46748] to the 5.3 branch.
Fixes #48355.

Location:

branches/5.3

Files:

• . (modified) (1 prop)
• src/js/media/views/settings.js (modified) (1 diff)

branches/5.3/src/js/media/views/settings.js

@@ -70 +70 @@
         // Handle button groups.
         } else if ( $setting.hasClass('button-group') ) {
-            $buttons = $setting.find('button').removeClass('active');
-            $buttons.filter( '[value="' + value + '"]' ).addClass('active');
+            $buttons = $setting.find( 'button' )
+                .removeClass( 'active' )
+                .attr( 'aria-pressed', 'false' );
+            $buttons.filter( '[value="' + value + '"]' )
+                .addClass( 'active' )
+                .attr( 'aria-pressed', 'true' );
 
         // Handle text inputs and textareas.