Excellent choice of colors, I would say!
Few remarks though...
The topleft logo (West islip...) is not as sharp as it could be. Maybe a different approach (I don't know, maybe put it around a shape or a graphic or something so it wouldnt look so squeezed).
You could also use CSS more, it will clean up your markup a lot.
For instance:
<td width="588" height="25" bgcolor="#FFFF99" style="border-style: none; border-width: medium">
<p align="center"><b><a href="spirits.htm#Vodka">
<font size="2">Vodka</font></a></b><font size="2"><font color="#800000"> /
</font><b><a href="spirits.htm#Gin">Gin</a></b><font color="#800000"> /
</font><b><a href="spirits.htm#Whiskey">Whiskey</a></b><font color="#800000"> /
</font><b><a href="spirits2.htm#Rum">Rum</a></b><font color="#800000"> /
</font><b><a href="spirits2.htm#Tequila">Tequila</a></b><font color="#800000"> /
</font><b><a href="spirits2.htm#Scotch">Scotch</a></b><font color="#800000"> /
</font><b><a href="spirits2.htm#Cordials">Cordials</a></b></font></td>
You could specify a div like this
# submenu a {text-weight: strong .....}
# submenu p {color: #800000 ....} (and other settings)
and the above code would look like this:
<td width="588" height="25" bgcolor="#FFFF99" id="submenu">
<p><a href="spirits.htm#Vodka">Vodka</a> /
<a href="spirits.htm#Gin">Gin</a> /
<a href="spirits.htm#Whiskey">Whiskey</a> /
<a href="spirits2.htm#Rum">Rum</a> /
<a href="spirits2.htm#Tequila">Tequila</a> /
<a href="spirits2.htm#Scotch">Scotch</a> /
<a href="spirits2.htm#Cordials">Cordials</a></td>
You could also do your rollovers with css, I was never too happy with fp rollovers.
Thats from the code view. The design on the other hand is just fine.
