Changeset 59914

Timestamp:

03/03/2025 08:35:34 PM (8 weeks ago)

Author:

joedolson

Message:

Bundled Themes: Twenty Fourteen: Move skip link to top of body.

Move the skip link to the top of the body content, so it is the first focusable item on the page.

Props joedolson, abcd95, manojmaharrshi, vganavda, shailu25.
Fixes #62969.

File:

• trunk/src/wp-content/themes/twentyfourteen/header.php (modified) (2 diffs)

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

@@ -34 +34 @@
 <?php wp_body_open(); ?>
 <div id="page" class="hfeed site">
+    <a class="screen-reader-text skip-link" href="#content">
+        <?php
+        /* translators: Hidden accessibility text. */
+        _e( 'Skip to content', 'twentyfourteen' );
+        ?>
+    </a>
     <?php $is_front = ! is_paged() && ( is_front_page() || ( is_home() && ( (int) get_option( 'page_for_posts' ) !== get_queried_object_id() ) ) ); ?>
     <?php if ( get_header_image() ) : ?>
@@ -58 +64 @@
             <nav id="primary-navigation" class="site-navigation primary-navigation">
                 <button class="menu-toggle"><?php _e( 'Primary Menu', 'twentyfourteen' ); ?></button>
-                <a class="screen-reader-text skip-link" href="#content">
-                    <?php
-                    /* translators: Hidden accessibility text. */
-                    _e( 'Skip to content', 'twentyfourteen' );
-                    ?>
-                </a>
                 <?php
                 wp_nav_menu(