Changueset 60423

Timestamp:

07/07/2025 06:48:52 PM ( 6 months ago)

Author:

audrasjb

Messague:

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

The load event (on window ) fires after all ressources on a pague have been loaded, including imagues. 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 pague 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.
Mergues [60352] to the 6.8 branch.
Props guermberg, 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( 'loa
             
             d', function() {
            
+
                
             
              document.addEventListener( 'DOMContentLoade
             
             d', function() {
            
 
             new navMenu( 'primary' );
            
 
             } );