Posted 04 January 2012 - 08:57 AM
Design
Initial impressions are good but on closer inspection there's a lot of tidying up to be done. I like the blue but I don't like the texture, I think it muddies and dirties the blue header. Crack open your PSD, remove the texture, let's see how it looks then.
Your CTA is very small and I'm not sure having it on the left hand side is the best placement. They generally go on the right hand side as this is where people most expect them to be.
Below the blue, it starts to get a little cluttered. I would have three boxes, not four, but keep them roughly the same size but more spaced out. I would give these three boxes more vertical spacing above and below.
Overall it's really not a bad attempt, it just suffers a bit by being a little too tight. Don't get too paranoid about squeezing everything in above the fold or whatever, scrolling is what people expect to do and the site would look a lot nicer if there was more vertical space between
Code
This is where you're clearly struggling. Your front end code is a complete shambles, there's no other way to put it.
For a start, you're using tables where you don't need to. If you want to be a forward-thinking designer and developer creating truly up to date websites then stop using tables.
Further, you're using inline styles. The only time you should ever do this is for styling emails. Put ALL your styling in your stylesheet - that's what it's there for.
To be honest, your front end code is so messy and so out of date I'd strongly advise you re-write all of it and switch to more up to date technologies such as HTML5. It's not hard, and you could probably re-code the whole site in a day.
Sure, it may not make much of a difference to the site to the end user, but if you're going to be selling websites, you need to make sure you did your own properly.
Good luck!