Pretty much everyone is pointing you the right way with the ‘lose the tables’ – you have not a single page on the entire site that has anything resembling tabular data.
From there you have other stuff to clean up/get rid of like CSS in the markup, use of tags that have NO place on a website after 2001 like CENTER, U and FONT – much less that none of your tag orders make the least bit of sense.
It would appear you are not grasping the two basic types of tags – block-level and inline-level. (which is NOT the same thing as CSS’ display:inline and display:block, though those are the default behaviors for ‘level’). Basically block level tags can contain almost any tag, while inline-level tags cannot be wrapped around block-level ones.
FONT, SPAN, I, B, EM, STRONG, A – inline-level.
DIV, LI, UL, P – block level.
So:
<ul id="navbar">
<b>
<!-- The strange spacing herein prevents an IE6 whitespace bug. -->
<font color="#8A360F">
<li><a href="index.html"><font color="#8A360F">Home</a></li>
Not only is using tags that have no business being on a website that is less than a decade old (FONT), you have a B and a FONT tag trying to wrap a LI – you can’t do that! At the bottom you have a <br> wedged between the </LI> and </UL> – you can’t do that either.
Also, certain tags can only DIRECTLY wrap certain other tags… For example the only valid child of a UL is LI, so you can’t put tags after UL and around LI – doesn’t work that way. Another is the one Atsea just pointed out – TABLE can only have as child elements THEAD, TBODY, TFOOT, CAPTION and TR. TR can only have as child elements TD and TH… while TD and TH can wrap just about anything.
Another example is this:
<font size="4">
<b><p><i>
That’s complete gibberish – again the tag that has no business even being used in modern markup, you have it and a bold tag trying to wrap a paragraph… invalid/improper markup.
You also have a LOT of unclosed tags. You open tags they need to be closed in the SAME order you opened them.
<center><h2><font color="#8A360F">************</h2></center>
You didn’t close FONT, FONT and CENTER have no business on a properly written page unless you take a trip in the wayback machine to 1997 – this one’s even wierder as what the devil makes a bunch of asterisks the heading for a section of content? That’s not a heading…
Even some of the simpler stuff is just waste – like the string of line-breaks doing padding’s job.
It may also help on your home page to turn it into meaningful content instead of it being a glorified splash page with nothing for screen readers, search engines or people browsing with images disabled to use… especially since that gives your keywords meta zero relevance to the page content, at which point it’s pretty much ignored by everything…
… and some consistency in your code formatting would help show all the times you fail to close tags like LI… and “image/title.ico” is NOT a valid mime-type.
… and copyright copyright? seems a little redundant.
Basically boils down to how you have 7.5k of markup doing 3.5k’s job.