Changeset 61626

Timestamp:

02/12/2026 07:33:19 PM (8 months ago)

Author:

joedolson

Message:

Media: A11y: Remove tabindex on tabpanels.

Tabindex should be set on a tabpanel if there is content in that tabpanel that is not focusable. The media library uses tabpanels, but the first content is either focusable or redundant.

Change the media library tabpanels to omit tabindex attributes.

Props alh0319, joedolson, westonruter, huzaifaalmesbah.
Fixes #63984.

File:

• trunk/src/js/media/views/media-frame.js (modified) (4 diffs)

trunk/src/js/media/views/media-frame.js

@@ -102 +102 @@
                         ariaLabelledby;
 
-                tabPanelEl.removeAttr( 'role aria-labelledby tabindex' );
+                tabPanelEl.removeAttr( 'role aria-labelledby' );
 
                 if ( this.state().get( 'menu' ) && this.menuView && this.menuView.isVisible ) {
@@ -112 +112 @@
                                         role: 'tabpanel',
                                         'aria-labelledby': ariaLabelledby,
-                                        tabIndex: '0'
                                 } );
                 }
@@ -128 +127 @@
                         ariaLabelledby;
 
-                tabPanelEl.removeAttr( 'role aria-labelledby tabindex' );
+                tabPanelEl.removeAttr( 'role aria-labelledby' );
 
                 // Set the tab panel attributes only if the tabs are visible.
@@ -138 +137 @@
                                         role: 'tabpanel',
                                         'aria-labelledby': ariaLabelledby,
-                                        tabIndex: '0'
                                 } );
                 }