First off, amazing work. I'm going to be using it to design my new site.

The title of the page already has the <h1> tag, as I believe it should. It might be better SEO to throw the site name inside a span - especially on the page.tpl.php. page-front.tpl.php might be able to leave it as <h1>

IMHO

Edited by: VeryMisunderstood; Added code tags around HTML to make this issue readable

Comments

drewschmaltz’s picture

drewschmaltz commented
Title: <span>Title</span> = > <span class="sitename">SiteName</span> » <h1>Title</h1> -> <span class="sitename">SiteName</span>

thanks verymisunderstood

stephthegeek’s picture

stephthegeek commented

Yep, this is already in the works. We keep all our themes on the same base code and this will be going into an upcoming release.

jwolf’s picture

jwolf commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.