I think the template is great, but the rest of the content hasn't been styled properly, however there is potential.
Your source isn't semantic, for instance a list of links should be contained in the list tags
<ul>
<li>Link1</li>
<li>Link2</li>
[...]
</ul>
Start thinking about what it is that you're displaying and then think which tags are the best to use.
The next step for you would be to take all styling out and use CSS to style it i.e. split content and style into seperate documents.
Try not to use tables to layout your page, it's bad semantics. Use div's to layout a page and then style them using CSS.
Make sure that all pages are formatted the same, so that the site has a sense of continuity, it's the worst thing browsing a site and thinking you've left the site you thought you were on, but all that's happened is that the style's different.
With regards to the JavaScript, you're only using it to extend the functionality of your page, which is great. Make sure your site doesn't depend on JS to work, as this makes it difficult for users with older/non-js browsers. However you've got multiple versions of the same rollover javascript, look into creating one and then have the function calls determine which image to 'rollover' i.e.
function swapon(what, to)
{
document.getElementByID(what).src="img/"+to;
}
function swapoff(what, to)
{
document.getElementById(what).src="img/"+to;
}
[...]
<a href="#" onMouseOver="JavaScript: swapon('img1','nav1r.gif');" onMouseOut="JavaScript: swapoff('img1','nav1.gif');"><img src="nav1.gif" id="img1"></a> [...]
[NB: Don't count on the above code working exactly, that's just off the top of my head ;)]
So in a nutshell, the site needs improving, but it's pretty good for a first site.