[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:  joemcgill
     Type:  defect (bug)    |      Status:  accepted
 Priority:  normal          |   Milestone:  Future Release
Component:  Media           |     Version:  4.4.2
 Severity:  normal          |  Resolution:
 Keywords:  needs-patch     |     Focuses:  performance
----------------------------+-----------------------------

Comment (by mikeschroder):

 Replying to [comment:23 codex-m]:
 > Thank you @mikeschroder for feedback and testing! I agree that
 `getImageColors()` will introduce a performance problem especially when
 re-sizing very large images. It is because processing time is directly
 proportional to the total number of pixels on the image.
 >
 > I'm currently testing a new patch which does not use this function while
 still solving the main issue and not introducing any new issues. It's
 looking good so far and will post it here very soon ;) I'm hoping we could
 introduce this in 4.7. For your update. Cheers.

 No problem!  Absolutely, if there is a patch before the deadline, it'll be
 considered for commit. Thanks for digging into this!

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