Changeset 46748

Timestamp:

11/18/2019 10:21:26 PM (5 years ago)

Author:

afercia

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".

Fixes #48355.

File:

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

trunk/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.