[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 bradyvercher):

 I disagree that the core function for rendering a video shortcode needs to
 be used to display a header video. They're two different use cases. One is
 content and the other is for appearance.

 Based on my experience, MediaElement.js generates a bunch of extra markup
 and overly specific styles that themes would need to account for and it'll
 needlessly bloat the header area. It will also likely introduce
 compatibility issues with plugins that extend or even replace the core
 shortcode with another library. Consider sites that add a pre-roll video
 or track stats for video shortcodes -- there's no reason that
 functionality needs to be enabled for a header video. As far as
 performance is concerned, the video shortcode method requires at least 4
 extra requests and 100KB in scripts and styles, plus however many other
 requests plugins might generate.

 For additional sources, there is a media workflow that allows for defining
 alternative formats and setting a poster image that could be useful here.

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