[wp-trac] [WordPress Trac] #38172: Enable Video Headers in Custom Headers

[WordPress Trac]

#38172: Enable Video Headers in Custom Headers
-----------------------------+------------------------------
 Reporter:  davidakennedy    |       Owner:
     Type:  feature request  |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Themes           |     Version:  trunk
 Severity:  normal           |  Resolution:
 Keywords:  needs-patch      |     Focuses:
-----------------------------+------------------------------

Comment (by celloexpressions):

 I think the fundamental idea is that a higher-level feature like theme
 support for video should leverage lower-level core functionality rather
 than implementing things separately. If we want to add a way for
 `wp_video_shortcode` to skip the MediaElement stuff, that would work, but
 building the video tag here feels wrong and fragments the core APIs for
 video.

 For what it's worth, I'm fairly neutral regarding whether or not we
 ''need'' MediaElement.js here. The browser compatibility aspects are nice,
 but the cross-browser consistency and the ability to skin (even to
 control-less youtube/vimeo) is a more compelling reason to use it. New
 core functionality should use existing core functionality rather than
 duplicating it.

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