Changeset 51967 for trunk/src/wp-content/themes/twentytwentyone/template-parts/header/site-nav.php

Timestamp:

11/01/2021 09:45:45 PM (3 years ago)

Author:

joedolson

Message:

General: Remove role="navigation" from nav elements.

Role="navigation" was required for assistive technology to recognize HTML5 element's native ARIA roles while HTML5 and ARIA were being introduced. With the deprecation of IE11, the role attribute is only required when mapping elements that don't have native role.

Props costdev, mukesh27.
Fixes #54054.

File:

• trunk/src/wp-content/themes/twentytwentyone/template-parts/header/site-nav.php (modified) (1 diff)

trunk/src/wp-content/themes/twentytwentyone/template-parts/header/site-nav.php

@@ -11 +11 @@
 
 <?php if ( has_nav_menu( 'primary' ) ) : ?>
-    <nav id="site-navigation" class="primary-navigation" role="navigation" aria-label="<?php esc_attr_e( 'Primary menu', 'twentytwentyone' ); ?>">
+    <nav id="site-navigation" class="primary-navigation" aria-label="<?php esc_attr_e( 'Primary menu', 'twentytwentyone' ); ?>">
         <div class="menu-button-container">
             <button id="primary-mobile-menu" class="button" aria-controls="primary-menu-list" aria-expanded="false">