Helpful criticism will only make you better so don't think I'm trying to be mean.
- It's broken in IE6.
- No clue why you've done the bottom padding/margin on your side column.
- You've done your headers as images and not in <h> tags. If you'd like to keep the images, you really should do image replacements on <h> tags. This way, search engines see them as headers, screenreaders see the text and normal users see the images as originally intended.
Code:
<h2 id="latestBlogs">Latest Blogs</h2>
h2#latestBlogs { display: block; width: **px; height: **px; background: url('../latestBlogs.jpg') no-repeat; text-indent: -9999px;}
- Not a big fan on the layout either. IMO, it looks quite dated. (once again, not trying to be mean, just being honest). As you're marketing yourself as a designer, I think you'd be better of with a layout that shows better design skills.
If you're quite happy with the layout, at the very least, please change the color of your side column header backgrounds. 'Latest Work' just looks misplaced.
- You have a 'valid css' link when your css brings up 1 error and 12 warnings. The error is just a mistyped ] character. The warnings are just for color and don't really matter but a client won't understand that. If they try validating, all they'll care about is the fact there are warnings.
To be perfectly proper, you should go through and give the offenders a background color. Depending on the layout, this may or may not be possible without ruining the layout. As such, I always just give them background: transparent;.
