Personally, I don't like the grey background - it sets a rather "down" mood for the site, and many pictures and colours will (and do) not look good against it. That said, the white text and orange links contrast effectively with it.
The contrast is less good on the menu bar, the white text being lost on the lightest grey. The way in which the background changes on hover is a bit ugly - I'd change just the foreground colour.
The text used throughout is a little small, and can't be re-sized in IE. Don't be afraid to use sub-headings, bullet points, colour etc. to break up your text and draw attention to different parts of it - most people scan web pages for the bits that interest them, rather than read it from top to bottom.
I'm not sure what the purpose is of the image in the top-right - the one with "Problem Solved" in the middle. If it doesn't have one, lose it.
The [tt]alt[/tt] attribute is supposed to have alternative text for people who can't, or don't want to, see your images. It should convey the same information as the image itself does. For your top-left logo the alt text should be "DSP Computer Services", i.e. the same wording as the image. For purely decorative images - like the ones on each side of the text - the correct usage is [tt]alt=""[/tt], explicitly telling screen readers that the image contains nothing they need to worry about.
Carefully proofread and spell-check your copy, for example:
This site has been set up to help show webites designed by DSP Computer Services
webites?
Your copy is pretty uniformly horrible. It's quite verbose at times:
This site has been set up to help show webites [sic] designed by DSP Computer Services, and offer that very service, web design.
I don't care why you set up the site. I don't want to hear about the site, I want to hear about
you, something like: "DSP Computer Services provides a top quality range of services in the fields of web design and custom PC construction" (OK, my copywriting skills aren't brilliant either, but you see the change in emphasis).
Sometimes your language in unnecessarily (or prematurely) technical. Say I'm a small businessman wanting a website, on your front page I see
DSP Computer Services specialises in PHP and CFM based websites
That sounds really complicated. I just want an ordinary website - maybe I'll go elsewhere. Similarly, on your Web Design page you talk about "W3 Standards", without explaining what they are or why I might care about them (note: most clients don't care about such things).
Try to look at the site from the perspective of a potential customer - what do they know? What do they want to know about you? What I want to know on your "Web Design" page is "what do you do and how much does it cost?" - what you give me is your portfolio (strange, because there's another page for that) and some technobabble.
On the PC Manufacture page, I suggest you spec out some actual PCs - processor, memory, cards, etc. and actual prices, stressing that they are examples and can be customised to meet exact requirements. You need to point up areas where you beat the competition - Why would I come to you, instead of Dell, or PC World?
Finally, the contact form. If you're looking to deal direct with the public, you need to include a postal address and other real-world information. You also need to think about what you ask for - why do I need to tell you my date of birth in order to get a quote? Mind your own business! An explicit notice saying that you're not going to sell my email address to spammers (or that you are) would be good for visitors not familiar with the Data Protection Act 1998 (UK)
Enough feedback for you?
-- Chris Hunt
Webmaster & Tragedian
Extra Connections Ltd