[wp-trac] [WordPress Trac] #50425: Do not lazy-load images which are very likely to be in the initial viewport

[WordPress Trac]

#50425: Do not lazy-load images which are very likely to be in the initial viewport
-------------------------------------------------+-------------------------
 Reporter:  flixos90                             |       Owner:  flixos90
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.5
Component:  Media                                |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests needs-dev-  |     Focuses:
  note                                           |
-------------------------------------------------+-------------------------
Changes (by azaozz):

 * version:   => trunk
 * type:  enhancement => defect (bug)


Comment:

 > My understanding is that modern browsers which implement native lazy
 loading would handle this decision making.

 Same here. Even if the browsers aren't super good at that at the moment,
 I'm pretty sure they will improve over time :)

 Looking at https://github.com/WordPress/wordpress-develop/pull/353, most
 of it makes sense. The only thing that would perhaps need a confirmation
 if it is buggy is this: https://github.com/WordPress/wordpress-
 develop/pull/353/files#diff-67a565933b9d6a2850abd1d7dc5585d5R1618 (do not
 lazy load images that come from `wp_get_attachment_image()` by default).

 In that terms thinking the PR should be added to trunk now except the
 above change. If it is confirmed as a bug during beta testing, lets add
 that too (changing ticket type to bug as that's what it is, essentially).

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/50425#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform