[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 has-unit-tests                        |  performance
-------------------------------------------------+-------------------------
Changes (by adamsilverstein):

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


Comment:

 Thanks again for all the details @noisysocks - I'll try combining the code
 from your PR with the unit tests from @pbearne and see if that passes.

 > As for WebP/AVIF, that doesn't solve the problem of reducing the palette
 back to the original number of colors. I tested WebP, and while it does
 admirably in some cases, it can still double or triple the file size
 without the palette reduction in place.

 Ah, right - that is the critical piece here, the palette reduction.

 I'll try giving this a test locally where I have several different
 versions of PHP/GD/Imagick running. Fine to fix just for Imagick for now,
 we can follow up with a GD fix.

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