[wp-trac] [WordPress Trac] #36477: Responsive images (srcset) can include images larger than the full size

[WordPress Trac]

#36477: Responsive images (srcset) can include images larger than the full size
-------------------------------------------------+-------------------------
 Reporter:  peterdavehello                       |       Owner:  pbearne
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  normal                               |   Milestone:  Future
                                                 |  Release
Component:  Media                                |     Version:  4.4.2
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-testing dev-         |     Focuses:
  feedback needs-unit-tests                      |  performance
-------------------------------------------------+-------------------------
Changes (by adamsilverstein):

 * keywords:  has-patch needs-testing dev-feedback has-unit-tests => has-
     patch needs-testing dev-feedback needs-unit-tests


Comment:

 @peterdavehello thanks for reporting this, very interesting.

 Thanks @nosilver4u for the detailed analysis and PR, I assume that
 replaces the two patched linked earlier in the ticket?

 These fixes are for Imagick - does GD suffer from the same issue, or have
 you tested this? Fixing both would make sense if possible.

 @peterdavehello have you considered using WebP as your output format (or
 even AVIF if your server supports it), I'm curious if that would get you
 smaller file sizes that way? Something like
 [https://gist.github.com/adamsilverstein/35e9d9fcdb1933e67ce38a7ccb119f3d
 this gist] should work.
 note: Adding "neeeds unit tests" tag

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