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

WordPress Trac noreply at wordpress.org
Fri Oct 14 14:20:06 UTC 2016


#38172: Enable Video Headers in Custom Headers
----------------------------+--------------------
 Reporter:  davidakennedy   |       Owner:
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  4.7
Component:  Themes          |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:
----------------------------+--------------------

Comment (by aaroncampbell):

 After reviewing the patch, my first thoughts were exactly the same as
 @mikeschroder. However, after reading through all the comments on here (I
 was largely convinced by @pento's data), I think the simple plain `video`
 tag is fine with the image fallback.

 There is one typo that I saw, just in a comment:
 > // If there was a video an it was removed, fall back to an image.
 Should be:
 > // If there was a video and it was removed, fall back to an image.

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


More information about the wp-trac mailing list