Well, you have a lot of empty space in the header.
The blue striped background does not match up well against the background in the header. You might want to try something that is a little more on the purple side.
The navigation with the underlining links gives the site an outdated look. You might want to try making the links bold and changing the background instead of underlining the links.
The coding isn't bad. You might want to remember that there are other html elements besides div and they have a purpose. For example the header tags (H1, H2, etc) are useful for people who use screen readers. They let the person know what is a header. Paragraph tags say there is a few lines of text to read. You can always change the way they look using CSS.
Last edited by baseiber; 05-03-2008 at 12:49 AM.
"An expert is a man who tells you a simple thing in a confused way in such a fashion as to make you think the confusion is your own fault."
- William Castle