Tagged: ,

Viewing 7 posts - 1 through 7 (of 7 total)
  • Author
    Posts
  • #16340

    Hi,

    Thanks for great themes. I was looking at the Coherence theme and was concerned about all the H1 headings that is on the homepage.

    Normally I would place the Logo as an H1 on the home page and then work my way down to h2, h3 etc, but on your demo as well on the site I am setting up which is pretty much the same configuration as your demo site, I ending up with way too many H1 that are competing for attentions.

    There are 8 H1 tags on the homepage:

    LOGO: Coherence 9

    WELCOME SLIDESHOW: A gourmet Theme 15

    WELCOME SLIDESHOW: This is a smaller caption with more text 40

    PORTFOLIO ITEM: KEREMIC 7

    PORTFOLIO ITEM: DISCOVER 8

    PORTFOLIO ITEM: EXPERIMENTS 11

    PORTFOLIO ITEM: IOS LOGO DESIGN 15

    Anyway without having to sift through the code too much to change that?

    #91143

    I would like to change the H1 “main-title” tag that is shown on my dynamic homepage when portfolio items are shown to H3, and I found the code in functions-coherence.php, but it is not working with is_home() nor is_frontpage().

    Is there a Avia framework call that I can use to see if the page is the homepage?

    if(is_front_page()) {

    $output .= ‘<h3 class=”main-title”>’.$title.’</h3>’;

    }

    else { $output .= ‘<h1 class=”main-title”>’.$title.’</h1>’;

    }

    …Mikael

    #91144

    Hi,

    I think it is “is_home”.

    Regards,

    Ismael

    #91145

    Hi Ismael,

    I have tried it with is_home() as well, and it it still serving up the <h1> tag on the homepage.

    I currently have modified the avia_title function in functions-choherence.php to the following

    if(is_home())

    {

    $output .= ‘<h3 class=”main-title”>’.$title.’</h3>’;

    }

    else

    { $output .= ‘<h1 class=”main-title”>’.$title.’</h1>’;

    }

    but it is not working… It must be due to the page being configured dynamically and built in the Template Builder, as I can use is_home function in the header and it works fine there.

    #91146

    Hi Mikael,

    All of the dynamic elements are created with functions in includes>helper-template-dynamic.php. You should be able to go through and change the h1 elements to h2 or h3 as you need.

    This may require some additional css fixes and changes since the current layout and structure is based on them having h1 in the selector.

    Regards,

    Devin

    #91147

    Devin,

    I just found the solution after looking into the loop-portfolio.php file, I could see that it was calling the avia_title function with the $class set as ‘portfolio-title’ and by adding a conditional check in avia_title I was able to fix the issue…..

    Previously the portfolio entries were coming up with <h1> tags in their titels on both the homepage, but as well in the portfolio page, but by adding this code, I have changed them to <h3>.

    if($class==’portfolio-title’)

    {

    $output .= ‘<h3 class=”main-title”>’.$title.’</h3>’;

    }

    else

    { $output .= ‘<h1 class=”main-title”>’.$title.’</h1>’;

    }

    #91148

    Hey,

    Glad you fixed it.

    Regards,

    Ismael

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

The topic ‘To many h1 Headings for SEO’ is closed to new replies.