From b51b969fd4fffe598f2f7ec61884b9fab0c5ba28 Mon Sep 17 00:00:00 2001
From: Caleydon Media <info@caleydon.com>
Date: Thu, 22 Nov 2012 16:45:01 +0100
Subject: [PATCH] Update themes/default/sitemap.tpl

SEO and accessibility improvement
- added link title and rel parameters
- improving dynamically displayed sitemap links (different list of link for logged/unlogged customer)
---
 themes/default/sitemap.tpl | 36 ++++++++++++++++++++++--------------
 1 file changed, 22 insertions(+), 14 deletions(-)

diff --git a/themes/default/sitemap.tpl b/themes/default/sitemap.tpl
index c83cffd26..92962c5ea 100644
--- a/themes/default/sitemap.tpl
+++ b/themes/default/sitemap.tpl
@@ -31,23 +31,31 @@
 	<div class="sitemap_block">
 		<h3>{l s='Our offers'}</h3>
 		<ul>
-			<li><a href="{$link->getPageLink('new-products')}">{l s='New products'}</a></li>
+			<li><a href="{$link->getPageLink('new-products')}" title="{l s='View a new products'}">{l s='New products'}</a></li>
 			{if !$PS_CATALOG_MODE}
-			<li><a href="{$link->getPageLink('best-sales')}">{l s='Top sellers'}</a></li>
-			<li><a href="{$link->getPageLink('prices-drop')}">{l s='Price drop'}</a></li>
+			<li><a href="{$link->getPageLink('best-sales')}" title="{l s='View a top sellers products'}">{l s='Top sellers'}</a></li>
+			<li><a href="{$link->getPageLink('prices-drop')}" title="{l s='View a products with price drop'}">{l s='Price drop'}</a></li>
 			{/if}
-			{if $display_manufacturer_link OR $PS_DISPLAY_SUPPLIERS}<li><a href="{$link->getPageLink('manufacturer')}">{l s='Manufacturers'}</a></li>{/if}
-			{if $display_supplier_link OR $PS_DISPLAY_SUPPLIERS}<li><a href="{$link->getPageLink('supplier')}">{l s='Suppliers'}</a></li>{/if}
+			{if $display_manufacturer_link OR $PS_DISPLAY_SUPPLIERS}<li><a href="{$link->getPageLink('manufacturer')}" title="{l s='List of a manufacturers'}">{l s='Manufacturers'}</a></li>{/if}
+			{if $display_supplier_link OR $PS_DISPLAY_SUPPLIERS}<li><a href="{$link->getPageLink('supplier')}" title="{l s='List of a suppliers'}">{l s='Suppliers'}</a></li>{/if}
 		</ul>
 	</div>
 	<div class="sitemap_block">
 		<h3>{l s='Your Account'}</h3>
 		<ul>
-			<li><a href="{$link->getPageLink('my-account', true)}">{l s='Your Account'}</a></li>
-			<li><a href="{$link->getPageLink('identity', true)}">{l s='Personal information'}</a></li>
-			<li><a href="{$link->getPageLink('addresses', true)}">{l s='Addresses'}</a></li>
-			{if $voucherAllowed}<li><a href="{$link->getPageLink('discount', true)}">{l s='Discounts'}</a></li>{/if}
-			<li><a href="{$link->getPageLink('history', true)}">{l s='Order history'}</a></li>
+		{if $cookie->isLogged()}
+			<li><a href="{$link->getPageLink('my-account', true)}" title="{l s='Manage my customer account'}" rel="nofollow">{l s='Your Account'}</a></li>
+			<li><a href="{$link->getPageLink('identity', true)}" title="{l s='Manage my personal information'}" rel="nofollow">{l s='Personal information'}</a></li>
+			<li><a href="{$link->getPageLink('addresses', true)}" title="{l s='List of my addresses'}" rel="nofollow">{l s='Addresses'}</a></li>
+			{if $voucherAllowed}<li><a href="{$link->getPageLink('discount', true)}" title="{l s='List of my discounts'}" rel="nofollow">{l s='Discounts'}</a></li>{/if}
+			<li><a href="{$link->getPageLink('history', true)}" title="{l s='List of my orders'}" rel="nofollow">{l s='Order history'}</a></li>
+		{else}
+			<li><a href="{$link->getPageLink('authentication', true)}" title="{l s='Authentication'}" rel="nofollow">{l s='Authentication'}</a></li>
+			<li><a href="{$link->getPageLink('authentication', true)}" title="{l s='Create new account'}"rel="nofollow">{l s='Create new account'}</a></li>
+		{/if}
+		{if $cookie->isLogged()}
+			<li><a href="{$link->getPageLink('index')}?mylogout" title="{l s='Log out'}" rel="nofollow">{l s='Log out'}</a></li>
+		{/if}
 		</ul>
 	</div>
 	<br class="clear" />
@@ -55,7 +63,7 @@
 <div id="listpage_content">
 	<div class="categTree">
 		<h3>{l s='Categories'}</h3>
-		<div class="tree_top"><a href="{$base_dir_ssl}">{$categoriesTree.name|escape:'htmlall':'UTF-8'}</a></div>
+		<div class="tree_top"><a href="{$base_dir_ssl}" title="{$categoriesTree.name|escape:'htmlall':'UTF-8'}">{$categoriesTree.name|escape:'htmlall':'UTF-8'}</a></div>
 		<ul class="tree">
 		{if isset($categoriesTree.children)}
 			{foreach $categoriesTree.children as $child}
@@ -70,7 +78,7 @@
 	</div>
 	<div class="categTree">
 		<h3>{l s='Pages'}</h3>
-		<div class="tree_top"><a href="{$categoriescmsTree.link}">{$categoriescmsTree.name|escape:'htmlall':'UTF-8'}</a></div>
+		<div class="tree_top"><a href="{$categoriescmsTree.link}" title="{$categoriescmsTree.name|escape:'htmlall':'UTF-8'}">{$categoriescmsTree.name|escape:'htmlall':'UTF-8'}</a></div>
 		<ul class="tree">
 			{if isset($categoriescmsTree.children)}
 				{foreach $categoriescmsTree.children as $child}
@@ -82,8 +90,8 @@
 			{foreach from=$categoriescmsTree.cms item=cms name=cmsTree}
 				<li><a href="{$cms.link|escape:'htmlall':'UTF-8'}" title="{$cms.meta_title|escape:'htmlall':'UTF-8'}">{$cms.meta_title|escape:'htmlall':'UTF-8'}</a></li>
 			{/foreach}
-			<li><a href="{$link->getPageLink('contact', true)}">{l s='Contact'}</a></li>
-			{if $display_store}<li class="last"><a href="{$link->getPageLink('stores')}" title="{l s='Our stores'}">{l s='Our stores'}</a></li>{/if}
+			<li><a href="{$link->getPageLink('contact', true)}" title="{l s='Contact'}">{l s='Contact'}</a></li>
+			{if $display_store}<li class="last"><a href="{$link->getPageLink('stores')}" title="{l s='List of our stores'}">{l s='Our stores'}</a></li>{/if}
 		</ul>
 	</div>
 </div>