Changeset 53100

Timestamp:

04/07/2022 10:12:34 PM (2 years ago)

Author:

audrasjb

Message:

Toolbar: Add a filter to help remove site icons from toolbar for large multisite, and lazy load them by default.

This changeset introduces the wp_admin_bar_show_site_icons filter to help developers to hide site icons from the toolbar, as it may have negative performance impact on large multisites. It also adds a default lazy load behavior for these icons.

Props wslyhbb, sabernhardt, lkraav, kebbet, peterwilsoncc.
Fixes #54447

File:

• trunk/src/wp-includes/admin-bar.php (modified) (1 diff)

trunk/src/wp-includes/admin-bar.php

@@ -612 +612 @@
     );
 
+
+
+
+
+
+
+
+
+
+
+
+
     foreach ( (array) $wp_admin_bar->user->blogs as $blog ) {
         switch_to_blog( $blog->userblog_id );
 
-        if ( has_site_icon() ) {
+        if ( has_site_icon() ) {
             $blavatar = sprintf(
-                '<img class="blavatar" src="%s" srcset="%s 2x" alt="" width="16" height="16" />',
+                '<img class="blavatar" src="%s" srcset="%s 2x" alt="" width="16" height="16" />',
                 esc_url( get_site_icon_url( 16 ) ),
-                esc_url( get_site_icon_url( 32 ) )
+                esc_url( get_site_icon_url( 32 ) ),
+                ( wp_lazy_loading_enabled( 'img', 'site_icon_in_toolbar' ) ? ' loading="lazy"' : '' )
             );
         } else {