Hay buddy I cant view any pages in IE7 so you may want to look at that.
Any way a few issues I noticed.
You have an issue where you have 2 versions of your site, this hurts your rankings in the search engines.
One version is under www. and the other is under the non www. version. To get round it you need to implament a 301 redirect.
To do that create a plain text file and name it .htaccess and place this code in it ~
RewriteEngine On
RewriteCond %{HTTP_HOST} !^www\.nikhandson\.co\.uk [nc]
RewriteRule ^(.*)$ http://www.nikhandson.co.uk/$1 [R=301,L]
Upload that file to your root directory.
You use a lot of spacer images.
<tr>
<td><img src="img/spacer.gif" width="398" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="127" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="71" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="13" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="76" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="13" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="93" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="11" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="48" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="14" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="62" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="14" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="41" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="19" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="1" height="1" border="0" alt="" /></td>
</tr>
Now it has been a while since I have needed to use a table layout, but I'm pretty sure you could replace all that with
<tr>
<td colspan="15" id="space"></td>
</tr>
#space{height:1px}
Also all your images can be reduced by placing them in CSS.
So for example rather than having
border="0"
Write ~
img{border:none}
or
a img{border:1px solid #FFF}
By doing that you would reduce a massive amount of code bloat and would only need a fraction of the code you have currently used.
I would personally replace your current html tag with
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
As that is more correct TBH.
You also have your entire navigation in images which is a bit mad really as they are just simple words and would be better spelt out for the search engines but as your alt attributes are currenlty blank all a blind user will here is
[link1]
[link2]
[link3]
When they should hear
[link1] showcase
[link2] cataloge etc....
But again this code ~
<tr>
<td rowspan="2" colspan="2"><img name="index_r2_c1" src="img/index_r2_c1.jpg" width="525" height="82" border="0" id="index_r2_c1" alt="" /></td>
<td><a href="showcase.html" target="contentframe1"><img name="linkshowcase" src="img/linkshowcase.jpg" width="71" height="16" border="0" /></a></td>
<td rowspan="2"><img name="index_r2_c4" src="img/index_r2_c4.jpg" width="13" height="82" border="0" id="index_r2_c4" alt="" /></td>
<td><a href="catalogue.html" target="contentframe1"><img name="linkcatalogue" src="img/linkcatalogue.jpg" width="76" height="16" border="0"></a></td>
<td rowspan="2"><img name="index_r2_c6" src="img/index_r2_c6.jpg" width="13" height="82" border="0" id="index_r2_c6" alt="" /></td>
<td><a href="photography.html" target="contentframe1"><img name="linkphotography" src="img/linkphotography.jpg" width="93" height="16" border="0"></a></td>
<td rowspan="2"><img name="index_r2_c8" src="img/index_r2_c8.jpg" width="11" height="82" border="0" id="index_r2_c8" alt="" /></td>
<td><a href="about.html" target="contentframe1"><img name="linkabout" src="img/linkabout.jpg" width="48" height="16" border="0"></a></td>
<td rowspan="2"><img name="index_r2_c10" src="img/index_r2_c10.jpg" width="14" height="82" border="0" id="index_r2_c10" alt="" /></td>
<td><a href="contact.html" target="contentframe1"><img name="linkcontact" src="img/linkcontact.jpg" width="62" height="16" border="0"></a></td>
<td rowspan="2"><img name="index_r2_c12" src="img/index_r2_c12.jpg" width="14" height="82" border="0" id="index_r2_c12" alt="" /></td>
<td><a href="links.html" target="contentframe1"><img name="linklinks" src="img/linklinks.jpg" width="41" height="16" border="0"></a></td>
<td rowspan="2"><img name="index_r2_c14" src="img/index_r2_c14.jpg" width="19" height="82" border="0" id="index_r2_c14" alt="" /></td>
<td><img src="img/spacer.gif" width="1" height="16" border="0" alt="" /></td>
</tr>
Would be better replaced with
<tr>
<td rowspan="2" colspan="14">
<ul>
<li><a href="showcase.html" target="contentframe1">Showcase</a></li>
<li><a href="catalogue.html" target="contentframe1">Catalogue</li>
<li><a href="photography.html" target="contentframe1">Photography</a></li>
<li><a href="about.html" target="contentframe1">About</a></li>
<li><a href="contact.html" target="contentframe1">Contact us</a></li>
<li>><a href="links.html" target="contentframe1">Links</a></li>
</ul>
</td>
</tr>
li{float:left;background:url(../img/index_r2_c12.jpg") no-repeat;padding-left:20px}
Maybe a bit of fidding would be needed but the amount of code would be hugely reduced which makes maintance so much easier.
Your title element should be Nik Handson : Graphic Designer in London
Fill in where you live as that would mean you would have a better chance of ranking for anyone looking for a graphic designer near you, no offence but I doubt you will rank in the top 10 for graphic design, and even if you did the people looking at graphic design would more than likley be graphic design students and not potential clients.
Some one looking for a graphic designer will search for graphic designer so that is the term you want to rank for.
Now TBH the other thing you are missing is a H1 tag, this is still very important from a SEO point of view. Now as you have your three images split up making you logo I would say that the best place to put it would be on this line ~
<td colspan="14"><img name="index_r1_c1" src="img/index_r1_c1.jpg" width="1000" height="20" border="0" id="index_r1_c1" alt="" /></td>
I would replace that with ~
<td colspan="14"><h1><img name="index_r1_c1" src="img/index_r1_c1.jpg" alt="Nic Henderson Graphic design" /></h1></td>
h1{width1000px;height:20px}
Then add all the rules under #index_r1_c1 in your CSS sheet to the end.
However I would rename it Nic Henderson Graphic designer based in London.
However to not be penalized in the search engines the image would need to say the same as well.
Not entirely sure why everything is named, that is a depricted tag and unless you need it for the Javascript which I haven't looked at they can all be removed.
All of that should only give you a few lines of code for your home page, better accessability, better keyword prominance for the search engines, faster load time, move effective rankings and a better chance to rank in higher search results positions.
With out seeing the other pages thats all I can comment on at the mo.
Jaz
Key:
Red ~ Mod_rewrite
Purple ~ XHTML
Green ~ Screen reader output
Blue ~ CSS
