May 21st, 2010, 06:44 AM
|
 |
guru
|
|
Join Date: Jan 2004
Location: SW1A London UK
Posts: 608
Time spent in forums: 3 Days 12 h 30 m 38 sec
Reputation Power: 0
|
|
looks good although quite a few things to focus on:
1) header and content not lining up.
The main header and the box aroudn the content do not line up.
It would be worth putting both into one DIV and giving the header and the content 100% width so you only have to resize the holder div:
e.g.
Code:
<div id="holder">
<div id="header"><!--HEADER HERE-->all your header stuff in here</div>
<div id="content"><!--CONTENT HERE-->content in here</div>
</div>
CSS =
#holder{ width:700px; overflow:auto}
#header{width:100%; background-color:red}
#content{width:100%; background-color:blue}
2) Content Margin:
I would suggest adding a suitable margin to your content (20px) so the text does not meet the border of the content. Same to be applied to the images(iStockphoto ones)
3) Javascript errors
Every page loads with javascript errors. Obviously you will need to sort those out before it is live.
4) Body Background image.
Personally i dont like the lines you have used and would change it to a gradient or something. But thats more personal then something actually wrong
Well done though
Ed
__________________
Create, Inspire & enjoy
|
Del.icio.us
Digg
Google
Spurl
Blink
Furl
Simpy
Y! MyWeb
Linear Mode
