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

Comment (by nosilver4u):

 The terminology is a bit confusing there, but what
 WP_Image_Editor_Imagick::get_png_color_depth() checks is typically called
 the "pixel depth" or "bits per sample". Valid values for pixel depth are
 1, 2, 4, 8, 16, and codeispoetry.png has a pixel depth of 8, color type 6
 (RGBA), which means 32 bits per pixel.
 http://www.libpng.org/pub/png/book/chapter08.html has more detail on all
 of that. I didn't see the 16-bit.png file in your fork, so I wasn't able
 to check that one. It can certainly be confusing, as I've seen people say
 "bit depth" in place of the "bits per pixel", but afaik the bit depth is
 how many bits are used for a single color/channel (R, G, B or A).

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