To many h1 Headings for SEO « Support Forum | Kriesi.at - Wordpress Themes and HTML Templates

Support Forum » Coherence

To many h1 Headings for SEO

7 posts from 3 voices

Started 8 months ago by zarako
Latest reply from Ismael
This topic is closed
This topic is resolved

Tags:

zarako
Member

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?

Posted 8 months ago #
zarako
Member

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

Posted 8 months ago #
Ismael
Moderator

Hi,

I think it is "is_home".

Regards,
Ismael

Posted 8 months ago #
zarako
Member

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.

Posted 8 months ago #
Devin
Moderator

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

Posted 8 months ago #
zarako
Member

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>';
}

Posted 8 months ago #
Ismael
Moderator

Hey,

Glad you fixed it.

Regards,
Ismael

Posted 8 months ago #