Sirrel Squirrel said:
Well as far as the HTML/CSS coding goes it is perfect according to W3 Schools Validator, I think if I just change the footer and background gradient it'll look a whole lot better, any other changes I can make?
The validator may say your code is correct, but ten seconds browsing through your HTML and I can pick out the following faults:
• Where are the headers? Your header image should not be inline (think of the mobile users!), it should be the background to a top-level header, defined in the CSS. Ditch the
div and you've just saved yourself a few bytes, as well as adding the ability to define a separate header image in a stylesheet designed specifically for different mediums (print, handheld etc.). Have a read up on the
Phark image replacement technique for pointers on why this method is better, and how you can do it.
• Again, ditch the
div surrounding the navigation unordered list. Nice job on using an unordered list for navigation, I must say, but there's no need for the extra
div.
• Put the first paragraph of text in
div#content in a paragraph, ditch the line-break. In an ideal world there should be absolutely no need for line-breaks — one should be able to make do with padding and margins.
• Add
title attributes to your anchor tags.
• The content in
div#footer would be better marked up as an unordered list, in my opinion.
...and I haven't even ventured into the CSS or commented on the design yet
*av