h1 logo - not good for SEO

i've just found that myStore template has all it's pages with the shop name as h1 tag which i think it's not good for SEO. Pages would have different h1 tag for each one. being a shop the h1 tag shoeld be the item name.

is there any way to remove h1 tag for logo?

in /gk_mystore/layouts/blocks/mainnav.php i found this code that generates the h1 tag but cant change it, every change a i make the logo disappear.

<h1 class="logo">
<a href="index.php" title="<?php echo $this->sitename(); ?>"><span><?php echo $this->sitename(); ?></span></a>
</h1>

Try this:

<div class="logo">
<a href="index.php" title="<?php echo $this->sitename(); ?>"><span><?php echo $this->sitename(); ?></span></a>
</div>

Then go to to file template.css and replace this code:

h1.logo,
div.logo-text { float:left; padding:0 10px 0 10px; background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; height:44px; position:relative; z-index:101; }
h1.logo { background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; }
h1.logo>a { text-indent:-999em; display:block; width:119px; height:44px; background:transparent url('../images/style1/logo.png') no-repeat 0 0; padding:0; }

with this code:

div.logo,
div.logo-text { float:left; padding:0 10px 0 10px; background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; height:44px; position:relative; z-index:101; }
div.logo { background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; }
div.logo>a { text-indent:-999em; display:block; width:119px; height:44px; background:transparent url('../images/style1/logo.png') no-repeat 0 0; padding:0; }

Let me know if it works.

Cheers

thx for the fast reply but it doesnt work. Now i've just got a link text with the shop name.


Edited:

Sorry, it's working like charm!! i had done sth wrong, sorry

thx for your help!!!!

OK..make also this changes in style1.css style2.css and style3.css.

Change this code (style1.css):

h1.logo,

h1.logo-text { background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; }

h1.logo { background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; }

h1.logo>a { background-image:url('../images/style1/logo.png'); }

with this:

div.logo,

div.logo-text { background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; }

div.logo { background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; }

div.logo>a { background-image:url('../images/style1/logo.png'); }

div.logo,

div.logo-text { background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; }

div.logo { background:#e3012d url('../images/style1/color_bg.png') repeat-x 0 bottom; }

div.logo>a { background-image:url('../images/style1/logo.png'); }


Do the same for style2 and style3 making the same changes.




Then open css3.css and make this changes:

h1.logo,
div.logo-text { -webkit-border-top-left-radius: 3px; -webkit-border-bottom-left-radius: 3px; -moz-border-radius-topleft: 3px; -moz-border-radius-bottomleft: 3px; border-top-left-radius: 3px; border-bottom-left-radius: 3px; }

with:

div.logo,
div.logo-text { -webkit-border-top-left-radius: 3px; -webkit-border-bottom-left-radius: 3px; -moz-border-radius-topleft: 3px; -moz-border-radius-bottomleft: 3px; border-top-left-radius: 3px; border-bottom-left-radius: 3px; }


It works for me.


Cheers.