[wp-trac] [WordPress Trac] #39893: Twenty Seventeen: Full Screen Video Header

[WordPress Trac]

#39893: Twenty Seventeen: Full Screen Video Header
---------------------------+------------------------------
 Reporter:  cingrosso      |       Owner:
     Type:  defect (bug)   |      Status:  new
 Priority:  normal         |   Milestone:  Awaiting Review
Component:  Bundled Theme  |     Version:  4.7.2
 Severity:  normal         |  Resolution:
 Keywords:                 |     Focuses:
---------------------------+------------------------------

Comment (by cingrosso):

 That's great Brady,

 I'll take a look and test the plugin and report back.

 '''Many thanks'''



 Replying to [comment:2 bradyvercher]:
 > I've looked into this a bit and there isn't really a great solution. The
 iframe that contains the video is automatically stretched to fill the
 screen, but YouTube has a resize handler that adjusts the dimensions of
 the video so that it always fits the screen rather than fill the iframe as
 desired.
 >
 > I did go ahead and [https://github.com/bradyvercher/custom-header-
 youtube put together a quick plugin] that attempts to counteract YouTube's
 behavior by resizing the iframe to ensure the video always fills the
 container. I haven't done much testing with Twenty Seventeen and it only
 works with widescreen videos at the moment, but otherwise it should be
 pretty solid.
 >

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