Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

Support Custom Headers and Video Headers? #243

Closed
celloexpressions opened this issue Oct 23, 2018 · 3 comments
Closed

Support Custom Headers and Video Headers? #243

celloexpressions opened this issue Oct 23, 2018 · 3 comments
Labels
enhancement New feature or request

Comments

@celloexpressions
Copy link

Let's consider adding support for custom headers. If we do, we could also support video headers.

The custom header could be off by default. Or, it could be on and include a compelling default image that ships with the theme. When used, it would match the display of featured images on single pages and posts (with the blue overlay). Perhaps it could be disabled in lieu of the featured image when a static front page is in use. This seems like a great opportunity to unify the single view and homepage view designs visually and functionally.

Video headers were added to core as part of the improvements with Twenty Seventeen. The theme-support implementation is very straightforward. This could act as a nice preview of future Gutenberg support for featured videos on individual posts and pages, and set Twenty Nineteen up for handling that type of feature design-wise.

@kjellr kjellr added the enhancement New feature or request label Oct 23, 2018
@kjellr
Copy link
Collaborator

kjellr commented Oct 23, 2018

There's an open PR to add custom header support (#133), but there's no issue for it, so this one can serve that purpose. 🙂

Video headers could be a nice enhancement, but I'd consider that somewhat low priority while we focus on tailoring the theme to current Gutenberg functionality. If someone would like to open a PR using code ported over from Twenty Seventeen though, I'll happily review and we can get that in.

@celloexpressions
Copy link
Author

I added the code for that to the header PR, since it's tied together as the same feature. Not sure what the preferred workflow would be to collaborate on that, but I tried to add all of the code with comment so that it's easy to add.

I would note that it's very easy to support video headers since the_custom_header_markup() is already being used. See https://make.wordpress.org/core/2016/11/26/video-headers-in-4-7/.

The only other step is to style the play and pause buttons - I will defer to the theme designer on the best approach for that.

@kjellr
Copy link
Collaborator

kjellr commented Jan 8, 2019

The PR related to this issue (#133) was closed a few weeks ago, so we could put the customization focus on Gutenberg, not features like custom headers. See @allancole's notes here: #133 (review)

As a result, I'm closing this issue as well. Thank you for your contribution!

@kjellr kjellr closed this as completed Jan 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants