Changeset 56556 for trunk/src/wp-content/themes/twentytwenty/functions.php

Timestamp:

09/12/2023 12:04:15 AM (3 years ago)

Author:

westonruter

Message:

Bundled Themes: Use defer loading strategy for theme scripts.

• Add defer loading strategy for all frontend end-user theme scripts (excluding Customizer preview).
• Move scripts to the head which relate to the initial page viewport to ensure they start loading earlier and execute sooner while still not blocking rendering.
• Update Twenty Twenty's script loader (TwentyTwenty_Script_Loader) to support core's built-in script loading strategies (#12009), while also retaining backwards-compatibility for child themes that may set async and defer script data.
• Update the main script loading strategy in Twenty Twenty from async to defer for better performance on repeat page views, since when an async script is cached it will block rendering.

Props westonruter, flixos90, sabernhardt.
Fixes #59316.

File:

• trunk/src/wp-content/themes/twentytwenty/functions.php (modified) (2 diffs)

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

@@ -136 +136 @@
          */
         $loader = new TwentyTwenty_Script_Loader();
-        add_filter( 'script_loader_tag', array( $loader, 'filter_script_loader_tag' ), 10, 2 );
+        if ( version_compare( $GLOBALS['wp_version'], '6.3', '<' ) ) {
+                add_filter( 'script_loader_tag', array( $loader, 'filter_script_loader_tag' ), 10, 2 );
+        } else {
+                add_filter( 'print_scripts_array', array( $loader, 'migrate_legacy_strategy_script_data' ), 100 );
+        }
 }
 
@@ -212 +216 @@
         }
 
-        wp_enqueue_script( 'twentytwenty-js', get_template_directory_uri() . '/assets/js/index.js', array(), $theme_version );
-        wp_script_add_data( 'twentytwenty-js', 'async', true );
+        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',
+                )
+        );
 }