[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:  joemcgill
     Type:  defect (bug)    |      Status:  reopened
 Priority:  normal          |   Milestone:  4.4
Component:  Media           |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:
----------------------------+------------------------

Comment (by joemcgill):

 I second everything @jaspermdegroot already mentioned. The main concern
 addressed in r35678 was to ensure we didn't add an additional `sizes`
 attribute whenever someone had already added a custom `sizes` attribute in
 the post editor.

 I agree with @azaozz that discouraging users from saving `sizes`
 attributes in the editor gives us the most flexibility going forward, but
 am not convinced that we ''should'' actively restrict this behavior. I
 also don't have strong feelings against restricting for now and giving
 ourselves some time to watch what issues come up over the next few months.

 That said, if restricting the use of standalone `sizes` attributes is the
 goal, would we be better off stripping those `sizes` attributes when the
 post is saved?

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