stojax777's Replies | WordPress.org

Forum Replies Created

Viewing 7 replies - 1 through 7 (of 7 total)

Forum: Themes and Templates
In reply to: [Twenty Seventeen] Mobile Menu Doesn’t Expand

Thread Starter stojax777

(@stojax777)

I am sorry, forgot to mention that it works on the provided address.
It stopped working on my local environment where I’m implementing the modifications to the website, before putting files on the production server.

Guess I’ll just strip down the wp-admin folder from the production server back to my local server.

Forum: Themes and Templates
In reply to: [Twenty Seventeen] Mobile Menu Doesn’t Expand

Thread Starter stojax777

(@stojax777)

9 years ago

Hi @riangraphics,

I actually don’t get to load the page at all. Only a blank screen.
I don’t understand how this happened, since this has nothing to do with the header.php that I modified.

This reply was modified 9 years ago by stojax777.
This reply was modified 9 years ago by stojax777.

Forum: Themes and Templates
In reply to: [Twenty Seventeen] Mobile Menu Doesn’t Expand

Thread Starter stojax777

(@stojax777)

9 years ago

Hi again,

I solved the custom header problem, but now I cannot log into the wordpress using wp-admin?

Anyone has any suggestions?

This how my header.php looks like:

<?php
/**
 * The header for our theme
 *
 * This is the template that displays all of the <head> section and everything up until <div id="content">
 *
 * @link https://developer.wordpress.org/themes/basics/template-files/#template-partials
 *
 * @package WordPress
 * @subpackage Twenty_Seventeen
 * @since 1.0
 * @version 1.0
 */

?><!DOCTYPE html>
<html <?php language_attributes(); ?> class="no-js no-svg">
<head>
<meta charset="<?php bloginfo( 'charset' ); ?>">
<meta name="viewport" content="width=device-width, initial-scale=1">
<link rel="profile" href="http://gmpg.org/xfn/11">

<?php wp_head(); ?>
</head>

<body <?php body_class(); ?>>
<div id="page" class="site">
	<a class="skip-link screen-reader-text" href="#content"><?php _e( 'Skip to content', 'twentyseventeen' ); ?></a>

	<header id="masthead" class="site-header" role="banner">
		
		<?php if ( is_front_page() ) : ?>
	<?php get_template_part( 'template-parts/header/header', 'image' ); ?>
	<?php endif; ?>
		

		<?php if ( has_nav_menu( 'top' ) ) : ?>
			<div class="navigation-top">
				<div class="wrap">
					<?php get_template_part( 'template-parts/navigation/navigation', 'top' ); ?>
				</div><!-- .wrap -->
			</div><!-- .navigation-top -->
		<?php endif; ?>

	</header><!-- #masthead -->

Forum: Themes and Templates
In reply to: [Twenty Seventeen] Mobile Menu Doesn’t Expand

Thread Starter stojax777

(@stojax777)

9 years ago

Thanks a lot @riangraphics,

I added the code to the header.php instead to site-branding.php and it totally solved my problem.

Forum: Themes and Templates
In reply to: [Twenty Seventeen] Mobile Menu Doesn’t Expand

Thread Starter stojax777

(@stojax777)

9 years ago

Hi again,

I went back through the template parts that I’ve changed and the only one was the site-branding.php

This was my code to show the custom-header and the call-to-action button that I created next to the logo.

<?php
/**
 * Displays header site branding
 *
 * @package WordPress
 * @subpackage Twenty_Seventeen
 * @since 1.0
 * @version 1.0
 */

?>
<?php if ( is_front_page() ) : ?>
<div class="site-branding">
	<div class="wrap">

		
        
        

		<div class="site-branding-text">
			 <?php if ( is_front_page() ) : ?>
				<h1 class="site-title"><a href="<?php echo esc_url( home_url( '/' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></h1>
			<?php else : ?>
				<p class="site-title"><a href="<?php echo esc_url( home_url( '/' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></p>
			<?php endif; ?>
            
			<?php $description = get_bloginfo( 'description', 'display' );
				if ( $description || is_customize_preview() ) : ?>
					<p class="site-description"><?php echo $description; ?></p>
				<?php endif; ?>
		</div><!-- .site-branding-text -->

        <?php if (is_front_page() ) : //This displays Book Now Button only on the front page?> 
        <div id="call-book-now">  <!-- this is a separate div created for a BOOK NOW button-->
        <?php the_custom_logo(); ?>
        <a href="#form">
            <button type="button" id="submit" class="special-button">RESERVE YOUR TABLE</button>
        </a>
                            
        </div>
        <?php else : ?>
        <!--continue the wrap div-->
        <?php endif; ?>
  
		<?php if ( ( twentyseventeen_is_frontpage() || ( is_home() && is_front_page() ) ) && ! has_nav_menu( 'top' ) ) : ?>
		<a href="#content" class="menu-scroll-down"><?php echo twentyseventeen_get_svg( array( 'icon' => 'arrow-right' ) ); ?><span class="screen-reader-text"><?php _e( 'Scroll down to content', 'twentyseventeen' ); ?></span></a>
	<?php endif; ?>

	</div><?php endif; ?><!-- .wrap -->
</div><!-- .site-branding -->

Now the menu works on every page, but I also have custom-header appearing on every page, any ideas how to get rid of the custom-header on other pages.

I tried through CSS class

body:not(.home-page).custom-header{
	visibility: hidden !important;
}

but that didn’t work out!

Please help!

Forum: Themes and Templates
In reply to: [Twenty Seventeen] Mobile Menu Doesn’t Expand

Thread Starter stojax777

(@stojax777)

9 years ago

Hi Andrew,
Thanks for a quick reply.

Can you please help me to find the template part which dislays other pages?
Should be the page.php?

I’ve realized what you’re saying, the HTML structure changes on the other pages, except the Homepage.
#masthead should be wrap around ‘navigation-top’ as it’s in the proper HTML that displays the Homepage, but I cannot figure out how is this happening since the code for the header #masthead is in header.php.
Supposedly it should be the same on every page?