Changeset 11448 for sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-learn-2020/header.php

Timestamp:

01/17/2022 04:45:59 AM (4 years ago)

Author:

dd32

Message:

Themes: Use a mu-plugin to output consistent skip-to links for all themes.

This is to remove the reliance upon $wporg_global_header_options.

See ​https://github.com/WordPress/wporg-mu-plugins/issues/42, ​https://github.com/WordPress/wporg-mu-plugins/pull/70.
Closes ​https://github.com/WordPress/wordpress.org/pull/49

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-learn-2020/header.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-learn-2020/header.php

@@ -14 +14 @@
 use function WPOrg_Learn\Locale\{ locale_notice };
 
+\WordPressdotorg\skip_to( '#main' );
+
 if ( FEATURE_2021_GLOBAL_HEADER_FOOTER ) {
     // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
     echo do_blocks( '<!-- wp:wporg/global-header /-->' );
 } else {
-    global $wporg_global_header_options;
-    if ( ! isset( $wporg_global_header_options['in_wrapper'] ) ) {
-        $wporg_global_header_options['in_wrapper'] = '';
-    }
-    $wporg_global_header_options['in_wrapper'] .= '<a class="skip-link screen-reader-text" href="#content">' . esc_html__( 'Skip to content', 'wporg-learn' ) . '</a>';
     wporg_get_global_header();
 }
@@ -37 +34 @@
 
 <div id="page" class="site">
-    <a class="skip-link screen-reader-text" href="#main"><?php esc_html_e( 'Skip to content', 'wporg-learn' ); ?></a>
-
     <div id="content">
         <header id="masthead" class="site-header <?php echo is_front_page() ? 'home' : ''; ?>" role="banner">