[FIX] Add ALT tag to sub-category images

[Martin]

This is a pretty simple fix and it's a bit of a mystery why it wasn't included sooner...

Anyway... This fix adds the all important ALT tag to your sub category images and resolves XML validation errors, as well as sorting out some basic accessibility issues.

For 5.x Open: /templates/__master/Snippets/SubCatItemImage.html
For 4.x Open: /templates/[your template]/Snippets/SubCatItemImage.html

Find:

Code: Select all

<img src="%%GLOBAL_SubCatImage%%" border="0" />

Replace with:

Code: Select all

<img src="%%GLOBAL_SubCatImage%%" border="0" alt="%%GLOBAL_SubCatName%%" />

[Painstik]

I have another ALT tag fix for product images for entire store

For 5.x Open: /lib/general.php

Find:

Code: Select all

$imageThumb .= '<img src="'.$thumb.'" alt="" />';

Replace with:

Code: Select all

$imageThumb .= '<img src="'.$thumb.'" alt="%%GLOBAL_ProductName%%" />';

[Martin]

This solution was posted by watersidedesigns on the Interspire forum here:
http://www.interspire.com/forum/showpos ... ostcount=7

I've just tidied it up into standard mod/edit format.

Open: /includes/display/ProductDetails.php

Find:

Code: Select all

				$thumbImage = '<img src="'.$thumb.'" alt="" />';
			}
			else if($thumb != '') {
				$thumbImage = '<img src="'.GetConfig('ShopPath').'/'.GetConfig('ImageDirectory').'/'.$thumb.'" alt="" />';

Replace with:

Code: Select all

				$thumbImage = '<img src="'.$thumb.'" alt="'.$GLOBALS['ProductName'].'" />';
			}
			else if($thumb != '') {
				$thumbImage = '<img src="'.GetConfig('ShopPath').'/'.GetConfig('ImageDirectory').'/'.$thumb.'" alt="'.$GLOBALS['ProductName'].'" />';

[scruffydoo]

I couldnt get this to work (or any of the alt tag mods), copied the code exactly and tried several times, I'm on 5.03, any kind soul help me out?

[Martin]

I think the mod from Painstik may have a bug in it as I suspect the %%GLOBAL_ProductName%% tag isn't translated... If it does, don't worry, but just in case here's a fix that might be required.

For 5.x Open: /lib/general.php

Find:

Code: Select all

$imageThumb .= '<img src="'.$thumb.'" alt="" />';

Replace with:

Code: Select all

$imageThumb .= '<img src="'.$thumb.'" alt="'.$GLOBALS['ProductName'].'" />';

[Martin]

I couldnt get this to work (or any of the alt tag mods), copied the code exactly and tried several times, I'm on 5.03, any kind soul help me out?

I can't find any reason for this to work any differently between 5.03 and the current version so it's likely the problem is more down to a syntax error caused by the editor being used or editee doing the editing.

Can you provide your modified file(s) as a ZIP attachment and I'll take a peek at them.

[irisjet]

I am getting a undefined index error when i try to add this, 5.0.5

[Martin]

I am getting a undefined index error when i try to add this, 5.0.5

Which modification and which file? There are three mods above so it would help to know which one is causing the issue.

Cheers.

[novista]

It would also be helpful to know where you are seeing that error message. If it is on a particular page, can you include a screen shot? There is another person reporting a similar problem.

[grantg]

I have made the above changes in 5.0.6 and thought this may be of interest to others, if it wasn't already obvious

Open includes/display/SideProductRelated.php

Find alt=""

Replace with: alt="'.$GLOBALS['ProductName'].'"


Now the Related Products also have an alt tag. I do wonder why this has been missed in ISC, especially because the alt="" tag was already there! Nevermind...maybe it'll get fixed sooner rather than later.