Like John says, not bad for a table-driven site. A few notes and pointers, in no particular order...
[ul]
[li]The first thing that drew my eye on the front page (well, I've actually only looked at the front page) is the blurry faces of the people on the strip of images across the top. Is there a reason for that? It looks weird to me.[/li]
[li]It would be nice to use real text instead of images for the menu across the top, you should be able to achieve a similar effect using CSS. Still, I'm glad to see you're using the correct [tt]alt[/tt] attributes on the menu images for those that can't see them.[/li]
[li]It would be good if the whole of the Our Cars/Apply Online/Finance Options link areas down the LHS were clickable, rather than just the title. I might try to click the little arrows or the brown text or the people (at least they're in focus this time!). This could easily accomplished using CSS and [tt]display:block[/tt] links.[/li]
[li]It's a personal pet hate, but I think the text size for the block of text in the middle of the screen is too small. You've got plenty of space to use a more comfortable size.[/li]
[li]Under the hood... You should really specify a full DOCTYPE (google for it), rather than the half one you're currently using. Be warned, you may need to change some code to stop your layout from breaking, but it will give you more consistent results across different browsers.[/li]
[li]This will not work:
Code:
<meta http-equiv="description" content="Duncton Specialist Vehicle Finance">
<meta http-equiv="keywords" content="Duncton Specialist Vehicle Finance">
You should be using [tt]name[/tt] instead of [tt]http-equiv[/tt] for description and keywords. [tt]http-equiv[/tt] is used to send extra http headers (which are better sent from the server if you can do so). Some of your other [tt]meta[/tt] elements are meaningless, but harmless.[/li]
[li]You're slightly over-using the [tt]alt[/tt] attribute. The intention is to provide alternative content for those who cannot see the images. In general, if the image contains text, the [tt]alt[/tt] should just be the text - for example, the [tt]alt[/tt] for your logo should be "Duncton", not "Duncton Logo" in my view. If I can't see it, why would I care whether it's a logo or just the word? If the image has no value other than decoration, use a [tt]alt=""[/tt]. I'd use this on the "tick" images down the left hand side - if I can't see them, I don't need to know about them.[/li]
[li]Table-based layout is a bit twentieth century, but I've seen a lot worse.[/li]
[/ul]
All in all, fairly minor stuff, with the exception of the lack of a full DOCTYPE. I didn't spot any stand-out SEO no-nos, though I'm sure there's stuff you can improve in that department (Isn't there always?)
Do you work in that converted barn? It looks really nice.
-- Chris Hunt
Webmaster & Tragedian
Extra Connections Ltd