Changeset 48749 for trunk/src/wp-includes/general-template.php

Timestamp:

08/07/2020 03:24:10 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Accessibility: Themes: Only unlink custom logo on the home page if the theme declares support for that.

To accommodate for the change, the custom-logo theme feature now accepts the unlink-homepage-logo parameter.

If and when the logo is linked on the home page, the link has the aria-current attribute for better accessibility.

Follow-up to [48039], [48283].

Props sabernhardt, audrasjb, johnstonphilip, demetris.
Fixes #37011.

File:

• trunk/src/wp-includes/general-template.php (modified) (3 diffs)

trunk/src/wp-includes/general-template.php

@@ -1022 +1022 @@
         );
 
-        if ( is_front_page() ) {
+        $unlink_homepage_logo = (bool) get_theme_support( 'custom-logo', 'unlink-homepage-logo' );
+
+        if ( $unlink_homepage_logo && is_front_page() && ! is_paged() ) {
             /*
              * If on the home page, set the logo alt attribute to an empty string,
@@ -1056 +1058 @@
         $image = wp_get_attachment_image( $custom_logo_id, 'full', false, $custom_logo_attr );
 
-        if ( is_front_page() ) {
+        if ( $unlink_homepage_logo && is_front_page() && ! is_paged() ) {
             // If on the home page, don't link the logo to home.
             $html = sprintf(
@@ -1063 +1065 @@
             );
         } else {
+            $aria_current = is_front_page() && ! is_paged() ? ' aria-current="page"' : '';
+
             $html = sprintf(
-                '<a href="%1$s" class="custom-logo-link" rel="home">%2$s</a>',
+                '<a href="%1$s" class="custom-logo-link" rel="home"%2$s>%3$s</a>',
                 esc_url( home_url( '/' ) ),
+                $aria_current,
                 $image
             );