Ticket #45424: 45424.13.patch

src/wp-content/themes/twentynineteen/functions.php

@@ -221 +221 @@
 
         wp_style_add_data( 'twentynineteen-style', 'rtl', 'replace' );
 
-        wp_enqueue_script( 'twentynineteen-skip-link-focus-fix', get_template_directory_uri() . '/js/skip-link-focus-fix.js', array(), '20151215', true );
-
         if ( has_nav_menu( 'menu-1' ) ) {
                 wp_enqueue_script( 'twentynineteen-priority-menu', get_theme_file_uri( '/js/priority-menu.js' ), array(), '1.0', true );
                 wp_enqueue_script( 'twentynineteen-touch-navigation', get_theme_file_uri( '/js/touch-keyboard-navigation.js' ), array(), '1.0', true );
@@ -237 +235 @@
 add_action( 'wp_enqueue_scripts', 'twentynineteen_scripts' );
 
 /**
+ * Fix skip link focus in IE11.
+ *
+ * This does not enqueue the script because it is tiny and because it is only for IE11,
+ * thus it does not warrant having an entire dedicated blocking script being loaded.
+ *
+ * @link https://git.io/vWdr2
+ */
+function twentynineteen_skip_link_focus_fix() {
+        // The following is minified via `terser --compress --mangle -- js/skip-link-focus-fix.js`.
+        ?>
+        <script>
+        /(trident|msie)/i.test(navigator.userAgent)&&document.getElementById&&window.addEventListener&&window.addEventListener("hashchange",function(){var t,e=location.hash.substring(1);/^[A-z0-9_-]+$/.test(e)&&(t=document.getElementById(e))&&(/^(?:a|select|input|button|textarea)$/i.test(t.tagName)||(t.tabIndex=-1),t.focus())},!1);
+        </script>
+        <?php
+}
+add_action( 'wp_print_footer_scripts', 'twentynineteen_skip_link_focus_fix' );
+
+/**
  * Enqueue supplemental block editor styles.
  */
 function twentynineteen_editor_customizer_styles() {
@@ -263 +279 @@
 
         require_once get_parent_theme_file_path( '/inc/color-patterns.php' );
 
-        if ( 'default' === get_theme_mod( 'primary_color', 'default' ) ) {
-                $primary_color = 199;
-        } else {
-                $primary_color = absint( get_theme_mod( 'primary_color_hue', 199 ) );
+        $primary_color = 199;
+        if ( 'default' !== get_theme_mod( 'primary_color', 'default' ) ) {
+                $primary_color = get_theme_mod( 'primary_color_hue', 199 );
         }
         ?>
 
-        <style type="text/css" id="custom-theme-colors" <?php echo is_customize_preview() ? 'data-hue="' . $primary_color . '"' : ''; ?>>
+        <style type="text/css" id="custom-theme-colors" <?php echo is_customize_preview() ? 'data-hue="' . absint( $primary_color ) . '"' : ''; ?>>
                 <?php echo twentynineteen_custom_colors_css(); ?>
         </style>
         <?php

src/wp-content/themes/twentynineteen/js/skip-link-focus-fix.js

@@ -3 +3 @@
  *
  * Helps with accessibility for keyboard only users.
  *
+ * This is the source file for what is minified in the twentynineteen_skip_link_focus_fix() PHP function.
+ *
  * Learn more: https://git.io/vWdr2
  */
 ( function() {