Originally posted by: CoinOperatedBoy
A few tips from running it through the w3c validator:
- You should declare your character encoding. Something like this:
<meta http-equiv="Content-Type" content="text/html;charset=utf-8" />
- You use a couple attributes, TOPMARGIN and LEFTMARGIN on the body tag. These aren't valid attributes. While you're at it, you should remove the link color attributes and toss them into your CSS (as you should with all style information).
- You should include ALT text attributes for all your images.
- All of your sidebar images use a self-closing tag style, which is not valid in HTML 4.01. You should either omit the forward slash from the end of the IMG tag or switch your document to XHTML.
- You shouldn't use the CENTER tag inside an inline element like TD. It's better to apply a style instead.
As for my own thoughts:
- Don't use tables for this layout, or just about any layout that doesn't specifically call for them. HTML/XHTML should be used to create some structure and hold content only, while CSS styles the page.
- You needn't have "themes" to justify having a separate CSS file. It makes it
easier to maintain, not harder.
- For your sidebar, you can do similar mouseover effects with CSS if you want to avoid Javascript since some people have it disabled. The :hover pseudo-class is made for this. Either way works, though. You might consider using a different "bone" graphic, maybe
something a little more realistic?
- As others have suggested, this might look better if it were centered.
- Sorry, but the whole masthead section looks pretty cheesy. The big Tahoma, mixing italics and bold, the logo... I understand some of this might be placeholder text, but I hope you're considering a big change up there. For example, the bark2paclogo.gif you have in the masthead is badly cut out / has some weird edge enhancement and looks pretty bad with the Arial and stencil typefaces. Don't mean to harp on the masthead, but if it's going to be present on every page and is the first thing anyone sees, it should be nice and clean.
- Other than that, just want to say it's cool you're doing this, and for free. Keep it up.