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

 @celloexpressions and @bradyvercher

 I didn't think of using `wp_video_shortcode()` although Brady has some
 good points about how themes and plugins modify it or would need to modify
 it.

 I went at this thinking a simple starting point was to use the HTML5
 `<video>` tag as is and just fallback to an image for anything that
 doesn't support it. It seems reasonable, very future friendly and less
 complex. The idea of easily skinning MediaElement.js is nice though. I do
 think controls will be needed for accessibility reasons.

 Explaining video encoding to users is frightening to me. :) I'm wondering
 if we can say it supports `.mp4` and that's it.

 Regarding the tie-in with header images – I imagined some kind of
 hierarchy. If video is there, it shows first, otherwise the image, etc.
 And I did think using header images would be smart for the `poster` and
 potentially for themers to wrap inside the video tag for a fallback too. I
 liked the idea of giving themers the file and letting them implement how
 they want to.

 Also, I'm not sure we need height and width even though I put it in. We
 probably should tie it into the content width in some way... I need to do
 more research there.

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