[wp-trac] [WordPress Trac] #34678: Responsive Images: Check if content images already have a sizes attribute

[WordPress Trac]

#34678: Responsive Images: Check if content images already have a sizes attribute
----------------------------+-----------------------
 Reporter:  jaspermdegroot  |       Owner:  azaozz
     Type:  defect (bug)    |      Status:  assigned
 Priority:  normal          |   Milestone:  4.4
Component:  Media           |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:
----------------------------+-----------------------
Changes (by joemcgill):

 * owner:  joemcgill => azaozz
 * status:  reopened => assigned


Comment:

 While I agree that it's not a good idea to add `sizes` attributes directly
 into the post editor—particularly without also pairing it with a `srcset`
 attribute—I don't think that we should actively restrict site owners who
 want to do this from being able to. If we find that a lot of people are
 using this "hack" to custom set `sizes` attributes, we should probably
 take that as a sign that our filter hooks need to be improved.

 On the other hand, if there is another reason to change the behavior
 introduced in r35678, (e.g., performance or other back-compat concerns) I
 think the approach in [attachment:34678.2.diff] is reasonable.

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