Changeset 56569

Timestamp:

09/13/2023 10:14:15 PM (15 months ago)

Author:

westonruter

Message:

Bundled Themes: Ensure Twenty Twenty's main script loads in head for WP<6.3.

Amends [56556].
Props westonruter, flixos90, peterwilsoncc.
Fixes #59316.

File:

• trunk/src/wp-content/themes/twentytwenty/functions.php (modified) (1 diff)

trunk/src/wp-content/themes/twentytwenty/functions.php

@@ -216 +216 @@
     }
 
-    wp_enqueue_script(
-        'twentytwenty-js',
-        get_template_directory_uri() . '/assets/js/index.js',
-        array(),
-        $theme_version,
-        array(
-            'in_footer' => false, // Because involves header.
-            'strategy'  => 'defer',
-        )
-    );
+    /*
+     * This script is intentionally printed in the head because it involves the page header. The `defer` script loading
+     * strategy ensures that it does not block rendering; being in the head it will start loading earlier so that it
+     * will execute sooner once the DOM has loaded. The $args array is not used here to avoid unintentional footer
+     * placement in WP<6.3; the wp_script_add_data() call is used instead.
+     */
+    wp_enqueue_script( 'twentytwenty-js', get_template_directory_uri() . '/assets/js/index.js', array(), $theme_version );
+    wp_script_add_data( 'twentytwenty-js', 'strategy', 'defer' );
 }