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

Timestamp:

11/04/2021 06:16:46 PM (3 years ago)

Author:

hellofromTonya

Message:

Bundled Themes: Remove the "role" attribute on HTML elements with a default landmark role.

With IE11 no longer supported, the primary problem with assistive technology support for native HTML5 elements no longer applies.

This commit removes the role attribute from the following HTML5 elements with default landmark roles, per formerly required role attributes and W3C.

Follow-up to [17669], [21261], [23452], [24832], [29892], [38833], [40851], [43842], [46271], [49216].

Props costdev, craigfrancis, joedolson, mukesh27, ryokuhi, sabernhardt.
Fixes #54079.

File:

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

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

@@ -14 +14 @@
 ?>
 
-<header id="masthead" class="<?php echo esc_attr( $wrapper_classes ); ?>" role="banner">
+<header id="masthead" class="<?php echo esc_attr( $wrapper_classes ); ?>">
 
     <?php get_template_part( 'template-parts/header/site-branding' ); ?>