Changeset 51967 for trunk/src/wp-content/themes/twentysixteen/header.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/twentysixteen/header.php (modified) (2 diffs)

trunk/src/wp-content/themes/twentysixteen/header.php

@@ -52 +52 @@
                     <div id="site-header-menu" class="site-header-menu">
                         <?php if ( has_nav_menu( 'primary' ) ) : ?>
-                            <nav id="site-navigation" class="main-navigation" role="navigation" aria-label="<?php esc_attr_e( 'Primary Menu', 'twentysixteen' ); ?>">
+                            <nav id="site-navigation" class="main-navigation" aria-label="<?php esc_attr_e( 'Primary Menu', 'twentysixteen' ); ?>">
                                 <?php
                                     wp_nav_menu(
@@ -65 +65 @@
 
                         <?php if ( has_nav_menu( 'social' ) ) : ?>
-                            <nav id="social-navigation" class="social-navigation" role="navigation" aria-label="<?php esc_attr_e( 'Social Links Menu', 'twentysixteen' ); ?>">
+                            <nav id="social-navigation" class="social-navigation" aria-label="<?php esc_attr_e( 'Social Links Menu', 'twentysixteen' ); ?>">
                                 <?php
                                     wp_nav_menu(