[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                                           |
-------------------------------------------------+-------------------------

Comment (by flixos90):

 Thanks for the feedback @azaozz and @adamsilverstein! I will update the PR
 for now as mentioned, so that `wp_get_attachment_image` keeps lazy-loading
 on by default. The improved support for opting out (if opting out, the
 recommendation is to use `null` or `false`, but not `'eager'`) plus the
 extra test coverage is useful to be merged already, regardless of the
 final decision here.

 I'll keep this ticket open so we can continue testing and discussing as
 needed.

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