Changeset 60423

Timestamp:

07/07/2025 06:48:52 PM (9 days ago)

Author:

audrasjb

Message:

Twenty Twenty-One: Add nav menu toggle behaviors at DOMContentLoaded instead of at the load event.

The load event (on window) fires after all resources on a page have been loaded, including images. This means that
on a slow connection, the mobile nav menu button may be rendered yet unresponsive to taps between when it first appears
and when the page has fully loaded. This is rectified by switching to the DOMContentLoaded event so that the nav menu
behaviors are attached as soon as the DOM has fully loaded.

Reviewed by audrasjb.
Merges [60352] to the 6.8 branch.
Props gernberg, westonruter, sabernhardt.
Fixes #63613.

Location:

branches/6.8

Files:

• . (modified) (1 prop)
• src/wp-content/themes/twentytwentyone/assets/js/primary-navigation.js (modified) (1 diff)

branches/6.8/src/wp-content/themes/twentytwentyone/assets/js/primary-navigation.js

@@ -212 +212 @@
     };
 
-    window.addEventListener( 'load', function() {
+    document.addEventListener( 'DOMContentLoaded', function() {
         new navMenu( 'primary' );
     } );