Changeset 56143 for trunk/src/wp-includes/media.php

Timestamp:

07/05/2023 09:36:23 PM (23 months ago)

Author:

flixos90

Message:

Media: Ensure that large images before the main query loop are counted towards lazy-loading threshold.

Following [55318], [55847], and [56142], certain images in the header of the page have received support for potentially receiving fetchpriority="high" and having the loading="lazy" attribute omitted. However, these images being present did not affect the "content media count" which counts the images towards a certain threshold so that the first ones are not lazy-loaded.

This changeset also increases that count for such header images if they are larger than a certain threshold. The threshold is used to avoid e.g. a header with lots of small images such as icon graphics to skew the lazy-loading behavior.

Props thekt12, spacedmonkey, flixos90.
Fixes #58635.

File:

• trunk/src/wp-includes/media.php (modified) (4 diffs)

trunk/src/wp-includes/media.php

@@ -5589 +5589 @@
         return $loading_attributes;
     };
+    // Closure to increase media count for images with certain minimum threshold, mostly used for header images.
+    $maybe_increase_content_media_count = static function() use ( $attr ) {
+        /** This filter is documented in wp-admin/includes/media.php */
+        $wp_min_priority_img_pixels = apply_filters( 'wp_min_priority_img_pixels', 50000 );
+        // Images with a certain minimum size in the header of the page are also counted towards the threshold.
+        if ( $wp_min_priority_img_pixels <= $attr['width'] * $attr['height'] ) {
+            wp_increase_content_media_count();
+        }
+    };
 
     $loading_attrs = array();
@@ -5641 +5650 @@
     $header_area = WP_TEMPLATE_PART_AREA_HEADER;
     if ( "template_part_{$header_area}" === $context ) {
+        // Increase media count if there are images in header above a certian minimum size threshold.
+        $maybe_increase_content_media_count();
         return $postprocess( $loading_attrs, true );
     }
@@ -5646 +5657 @@
     // The custom header image is always expected to be in the header.
     if ( 'get_header_image_tag' === $context ) {
+        // Increase media count if there are images in header above a certian minimum size threshold.
+        $maybe_increase_content_media_count();
         return $postprocess( $loading_attrs, true );
     }
@@ -5672 +5685 @@
             && did_action( 'get_header' ) && ! did_action( 'get_footer' )
         ) {
+            // Increase media count if there are images in header above a certian minimum size threshold.
+            $maybe_increase_content_media_count();
             return $postprocess( $loading_attrs, true );
         }