New website! What do you think

[janobar]

Tell me what you think guys, just finished it and wondering if anyone likes it.
http://www.jeremydonovan.com.au

[Kitchensink108]

Looks good; really nice, design-wise. I'd make the nav links bigger, to fill the space better (either that or move the links down 2-3px so they're centered better).

The link previews are good, though I wasn't sure why the links in the Links section didn't have them.

Probably the most surprising thing was that the code was actually decent. There are a few ways to make it a bit better:
-some of your meta tags are extraneous (in some people's eye, they're all extraneous besides Content-Type)
-You have a lot of attributes (mainly heights and widths) that would be better defined in CSS than HTML.
-In XHTML, use the <strong> tag instead of the <b> tag
-If you have an element with a className, you don't need to give every child element a className (referring to BulletList/BulletLink).
-div id="warpper". Is that supposed to be "wrapper"? Not that it matters, I'm just curious.

I also checked the sizes of your files with:
http://www.websiteoptimization.com/services/analyze/
I think it came in at about 160kb, with 23 images. imo, that's acceptable, but just make sure you compress each image individually to optimize download speeds.

[janobar]

Thanks for some good advice and as you probably know I am still learning this stuff and all suggestions are really important for me. I used "widths" an "heights" in html to be valid WC3. What's the advantage of using <strong> in CSS over <b> in HTML tag?
Thanks again.
http://www.jeremydonovan.com.au

[AJain]

<i> - Strictly a presentational element.
If you want text to be italicized and not have any real semantic meaning, then wrap it in <i></i>.

<b> - Strictly a presentational element.
If you want text to be bold and not have any real semantic meaning, then wrap it in <b></b>.

<em> - Strictly a structure element.
If you want text to be spoken with emphasis using a speech reader, then wrap it in <em></em>.

<strong> - Strictly a structure element.
If you want text to be spoken with strong emphasis using a speech reader, then wrap it in <strong></strong>.

*Note: The above was taken from a webmasterworld forum post.

HTML was designed to structure content and nothing more, CSS was introduced to work with only the presentation of the document.

So if you find yourself adding &nbsp; all over the place to space an item you are using HTML to fix the presentation which should be handled by CSS ;)

[Kitchensink108]

<b> is deprecated in XHTML. It's still supported by all browsers, but for the sake of good code, you should switch to <strong>.

[janobar]

Very interesting stuff thanks guys! I didn't know all this and never thought about coding for the purposes of using a speech reader. This is really something! Thanks again.
http://www.jeremydonvan.com.au
http://www.antypody.com