The look of the site is nice. I think I would make the menu stand out a shade bit more, but that's about it. Code wise is a different beast. Your site isn't W3C valid. You use a table based layout. You also use what I call "CSS for Dummies." In other words, the only CSS us really use is for links and the scroll bar. Take your JavaScript and CSS and link them externally. Then start to add CSS in your external file as you go through and trim down your code. Do the easy stuff first like getting rid of <font> tags and such. I wouldn't doubt that your markup code would be cut in half at the minimum when you are all done.
Read "Designing with Web Standards" for a good starting place.
Bookmarks