Changeset 45300 for trunk/src/wp-admin/includes/ms.php

Timestamp:

05/09/2019 08:57:06 PM (6 years ago)

Author:

afercia

Message:

Accessibility: Add aria-current to the links in the <nav> elements introduced in [44905].

Also fixes a few translators comments and coding standards.

Props audrasjb.
Fixes #47106.

File:

• trunk/src/wp-admin/includes/ms.php (modified) (2 diffs)

trunk/src/wp-admin/includes/ms.php

@@ -1119 +1119 @@
         $classes = array( 'nav-tab' );
 
+        // Aria-current attribute.
+        $aria_current = '';
+
         // Selected is set by the parent OR assumed by the $pagenow global
         if ( $r['selected'] === $link_id || $link['url'] === $GLOBALS['pagenow'] ) {
-            $classes[] = 'nav-tab-active';
+            $classes[]    = 'nav-tab-active';
+            $aria_current = ' aria-current="page"';
         }
 
@@ -1131 +1135 @@
 
         // Add link to nav links
-        $screen_links[ $link_id ] = '<a href="' . esc_url( $url ) . '" id="' . esc_attr( $link_id ) . '" class="' . $esc_classes . '">' . esc_html( $link['label'] ) . '</a>';
+        $screen_links[ $link_id ] = '<a href="' . esc_url( $url ) . '" id="' . esc_attr( $link_id ) . '" class="' . $esc_classes . '"' . $aria_current . '>' . esc_html( $link['label'] ) . '</a>';
     }