Changeset 49861

Timestamp:

12/21/2020 06:49:22 PM (4 years ago)

Author:

desrosj

Message:

Twenty Twenty-One: Do not specify loading=“eager” for single post thumbnails.

While loading=“eager” is a supported alternative to omitting the attribute entirely, browsers follow the value of this attribute explicitly when specified.

Specifying eager would prevent the user from receiving any additional potential benefits implemented at the browser level, such as further mechanisms to automatically decide which elements to lazy-load.

Props flixos90, ryelle, poena.
Merges [49860] to the 5.6 branch.
Fixes #52139.

Location:

branches/5.6

Files:

• . (modified) (1 prop)
• src/wp-content/themes/twentytwentyone/inc/template-tags.php (modified) (1 diff)

branches/5.6/src/wp-content/themes/twentytwentyone/inc/template-tags.php

@@ -191 +191 @@
             <figure class="post-thumbnail">
                 <?php
-                // Thumbnail is loaded eagerly because it's going to be in the viewport immediately.
-                the_post_thumbnail( 'post-thumbnail', array( 'loading' => 'eager' ) );
+                // Lazy-loading attributes should be skipped for thumbnails since they are immediately in the viewport.
+                the_post_thumbnail( 'post-thumbnail', array( 'loading' => false ) );
                 ?>
                 <?php if ( wp_get_attachment_caption( get_post_thumbnail_id() ) ) : ?>