independent-publisher icon indicating copy to clipboard operation
independent-publisher copied to clipboard

Feature Request: Option to show header + nav menu on single post pages

Open joeross opened this issue 10 years ago • 8 comments

This is something I want on mobile, because I want readers who accidentally tap a link to a post page, or who come in the "side door" from a social link, to be able to easily return to the home page without scrolling to the bottom.

joeross avatar Oct 17 '14 13:10 joeross

@joeross Thanks for the feature request! This sounds like a great option to have. I'll accept this feature request and add it to the todo list of the next release. :)

raamdev avatar Oct 17 '14 18:10 raamdev

I'm bumping this off the Next Release milestone because there's already an option to show the Navigation Menu on Single Posts (Appearance → Customize → General Options → Show Nav Menu on Single Posts).

I'm leaving this issue open in case someone wants to submit a Pull Request to add this option.

raamdev avatar May 02 '16 15:05 raamdev

@raamdev thank you for awesome theme!

+1 this request, also it will be really cool to show header image on Single pages in mobile view. Yep, there is an option to show Navigations menu that actually allows to create a link to Home page, but:

  1. On non-mobile views a link to "Home" page in menu looks strange, 'cause there's a header image, which already serves this purpose
  2. On mobile view a header without image // logo looks empty. It's even hard to recognise it's a Navigation menu

That't complete IMHO, however seems reasonable for me.

f1nnix avatar Aug 24 '16 14:08 f1nnix

@raamdev Agreed; this is an amazingly good theme. Want to bump this issue.

I have the option to show the nav menu on single pages enabled, and that works fine.

It would be very useful, for all the reasons above, to display independent_publisher_site_info() on single pages. I looked at header.php but even removing the test for is_single() could not get that information to display on a single page.

Any help or guidance gratefully received.

jeremycherfas avatar Apr 28 '17 17:04 jeremycherfas

I don't know if this will make it into the next release, but it does look quite simple to do. I spent some time thinking a bit more about this and I wanted to document what I think the new option should be called (this is only a mockup—this feature has not yet been implemented):

2017-04-28_14-57-00


@jeremycherfas writes...

I looked at header.php but even removing the test for is_single() could not get that information to display on a single page.

There's some CSS that hides the div containing that extra information when the browser width is less than 1200px:

@media only screen and (max-width: 1200px) {
     .single .site-header-info {
          display: none;
     }
}

Also note that by default header.php uses independent_publisher_posted_author_card() when showing a Single post, so that will also need to be modified that so that it uses independent_publisher_site_info().

raamdev avatar Apr 28 '17 19:04 raamdev

Thanks @raamdev I think the mockup looks very clear.

I hadn't spotted the bit of magic hiding that div, so I am happy you pointed it out. I going to continue plugging away at my little tweaks today.

jeremycherfas avatar Apr 29 '17 10:04 jeremycherfas

In the end the simplest solution was to change site-header-info to my-siteheader-info. That enables me to easily make it available on all pages and avoid problems with the breakpoints. I checked everything I could, and nothing else seems to be broken. Thanks again for your help.

jeremycherfas avatar Apr 29 '17 13:04 jeremycherfas

@raamdev First of all Hi and thanks for the great theme! I also would like to have a full site-header on posts that don't contain a featured image. I think that would be a great addition! All the best!

svnsbck avatar Jun 25 '17 19:06 svnsbck