Vantage 1.3.1 - Double IDs and H1 Logo
Hi Dimitris,
just a couple of very little fixes.
1. The "adv_categories" ID is repeated twice. One for the "Listings Categories" menu item and then again for the "Event Categories" menu item. Just want to be totally HTML standard.
2. The logo structure is as follows:
Code:
<hgroup>
<h1 id="site-title">
<a style="height:70px;width:400px;background: transparent url('http://demos.appthemes.com/vantage/wp-content/themes/vantage/images/vantage-logo.png') no-repeat 0 0;" class="custom-header-image" href="http://demos.appthemes.com/vantage/">Vantage Demo</a>
</h1>
<h2 style="color:#444444;" id="site-description">A business directory application theme</h2>
</hgroup>
Based on the W3C HTML5 Validator (
see here) the hgroup tag is not allowed to be a child of a div element.
3. Having the logo inside an H1 will make us have an additional h1 on every page. So instead of just having the article, post, listing or event's H1, we'll have another h1 with the logo. I don't think this is good for
SEO. A listing should have one h1, one title.
Thanks for thaking a look into this.