chilipie
09-25-2004, 03:43 PM
Please comment on my first entirely valid XHTML & CSS design, for my web design company.
The site is located at:
http://www.redblizzard.tk/
http://users.quickfox.org/~chilipie/redblizzard/ <<< LINK NO LONGER VALID >>>
http://spyderz.port5.com/
ReadMe.txt
09-25-2004, 06:47 PM
i would recommend a little more padding on the main body section and possibly a min-height: 400px; you could do an IE hack to simulate this since IE treats height as min-height, but i wouldnt bother:
height: 400px;
min-height: 400px;
}
html>body selector {
height: auto;
}
the source itself isn't too bad but there are a few mistakes,
<div id="header"></div> is bad xhtml as it's meaning less, use an H1 and image replacement to achieve the same effect with graceful degradation.
The menu should also me marked up as an unordered list since thats what it is - a list of links.
Another point is that there aren't actually any heading tags at all - a <h2> saying what each page is at the top of the content block might be handy.
chilipie
09-26-2004, 08:18 AM
Thank you - I have added more padding and added the "min-height" attrribute to the content area, and have added a header to each page, and will shortly upload it. What is the H1 image replacement though? Also I am not sure how to make the unordered list get rid of the bullets and be horizontal rather than vertical.
Thanks again,
Ol
I forgot to say thanks to gohankid77 and Antoniohawk for their help with my layout on the HTML & CSS forum :p
ronaldb66
09-28-2004, 08:34 AM
You can read all about the various list treatments in the Listutorial (http://css.maxdesign.com.au/listutorial/index.htm) from MaxDesign; if you Google for "image replacement (http://www.google.com/search?hl=en&ie=UTF-8&q=image+replacement&btnG=Google+Search)" you'll find virtually every IR technique and variations published up to date.
chilipie
10-02-2004, 08:39 AM
Thanks for the links ronaldb66 :thumbsup:. I've used the image replacement "thingy", added a header to each page, made the menu into a list, put extra padding on the content area and added the min-height attribute to my CSS - so any other comments would be nice :p .