Make WordPress Core

Opened 8 years ago

Last modified 2 weeks ago

#39623 new enhancement

Twenty Seventeen: Consider removing custom header from internal pages if selected video header

Reported by: karmatosed's profile karmatosed Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: needs-refresh has-patch
Focuses: Cc:

Description

This came up as I first off didn't even notice I still had default custom header until went to that blog page on the front end.

It felt logical to me that the video header I had set would continue in the main site or at least I'd not still see the default header image.

Attachments (1)

39623.diff (2.7 KB) - added by xkon 7 years ago.
Option to apply video header in internal pages

Download all attachments as: .zip

Change History (4)

@xkon
7 years ago

Option to apply video header in internal pages

#1 @xkon
7 years ago

This made sense to me as well after building a couple of websites with 2017 that might have the need to use the video everywhere.

39623.diff adds an extra option under the Theme Options in Customizer to apply the video in all pages instead of only the front page. Some users might still want the video on the front page only that's why I made it as an extra option instead of forcing it.

It is working ( the video is applied everywhere ) but the CSS is missing as that part got me a bit confused, if someone wants to tackle the CSS changes required for this it would be awesome.

Also a question came up when I was doing this: on the internal pages there is an extra 'header' with the featured image. Should that stay as is? Or if the video is to be played everywhere we should also cancel the featured image header ? I'm asking because if you see a full screen video + the header featured image the scroll down to content would be long I imagine.

(Note: For my websites purposes I have added extra css with a not-so-proper-way to bypass the default styling and have the video applied properly + hidden the featured image completely if the video is applied. )

#2 @poena
7 months ago

  • Keywords needs-refresh added

#3 @karmatosed
2 weeks ago

  • Keywords has-patch added
Note: See TracTickets for help on using tickets.