[wp-trac] [WordPress Trac] #60178: wp_video_shortcode() outputs invalid HTML

[WordPress Trac]

#60178: wp_video_shortcode() outputs invalid HTML
---------------------------------------------+-----------------------
 Reporter:  jongycastillo                    |       Owner:  audrasjb
     Type:  defect (bug)                     |      Status:  closed
 Priority:  normal                           |   Milestone:  6.8
Component:  Media                            |     Version:
 Severity:  normal                           |  Resolution:  fixed
 Keywords:  has-patch has-unit-tests commit  |     Focuses:
---------------------------------------------+-----------------------
Changes (by audrasjb):

 * status:  accepted => closed
 * resolution:   => fixed


Comment:

 In [changeset:"59954" 59954]:
 {{{
 #!CommitTicketReference repository="" revision="59954"
 Media: Improve HTML5 compliance of `wp_video_shortcode()` boolean
 attributes.

 This changeset updates `wp_video_shortcode()` to improve boolean
 attributes handling in accordance with HTML5 standards. Technically, it
 replaces `attr="1"` with `attr` for the `loop`, `autoplay` and `muted`
 attributes. The `preload` attribute is also updated to accept only allowed
 values: `none`, `metadata`, and `auto`. If a value outside of this list is
 provided, it will be ignored, preventing invalid attribute outputs.

 Props jongycastillo, sabernhardt, joedolson, audrasjb, shub07,
 debarghyabanerjee.
 Fixes #60178.
 }}}

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