[wp-trac] [WordPress Trac] #35045: Responsive images not added when effective scheme differs from image src scheme

WordPress Trac noreply at wordpress.org
Wed Dec 16 04:09:41 UTC 2015


#35045: Responsive images not added when effective scheme differs from image src
scheme
-------------------------------------------------+-------------------------
 Reporter:  webaware                             |       Owner:
     Type:  defect (bug)                         |  johnbillion
 Priority:  normal                               |      Status:  reviewing
Component:  Media                                |   Milestone:  4.4.1
 Severity:  normal                               |     Version:  4.4
 Keywords:  has-patch needs-testing has-unit-    |  Resolution:
  tests                                          |     Focuses:  template
-------------------------------------------------+-------------------------

Comment (by joemcgill):

 > Was also thinking we can move the "URL exists in image meta" check to
 wp_calculate_image_srcset()

 I was thinking about this again earlier today while discussion #35102.
 Moving this logic to `wp_calculate_image_srcset()` streamlines the code a
 little bit and has the added benefit allowing the filter to still be
 reachable by plugin authors who have rewritten image URLs for whatever
 reason (e.g., CDNs, etc.).

 [https://core.trac.wordpress.org/attachment/ticket/35045/35045.4.diff
 35045.4.diff] is a first pass at what this would look like if we moved the
 logic from
 [https://core.trac.wordpress.org/attachment/ticket/35045/35045.2.diff
 35045.2.diff] into `wp_calculate_image_srcset()`.

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


More information about the wp-trac mailing list