On Thu, 17 Aug 2006 04:44:59 -0700, TomB wrote:
It's really not bad at all, so these are just knit picks:
_____________________
<div id="foot">
<span>Copyright by DUSK 2006 - All rights reserved.</span>
<span><img src="img/icon_mini.gif" alt="*"></span>
<span><a href="" name="Site administration">Site Admin</a></span>
<span><img src="img/icon_mini.gif" alt="*"></span>
<span><a href="mailto:duskadmin (AT) gmail (DOT) com">Contact webmaster</a></span>
</div>
There must be a better way to do this without the span-itis. Since the
only thing that <span> does is { margin-left: 10px; }, you could simply
style the "foot" div with selectors something like
#foot a img span{
margin-left: 10px;
}
____________________
I find these two lines bizarre:
html { height: 95%; }
body { height: 85%; }
....and don't know what real use something like this would be, except
to provide padding around an overflow:auto scrolling div that should
just be allowed to grow and scroll normally with the page. It just seems
a bit baroque to me, but not necessarily an error.
____________________
you have your navigation font size set to 85%, which is definitely not a
good idea.
_____________________
Your CSS needs some cleaning up, according to the validator:
* Line: 39 Context : p
Parse Error - 2em;
* Line: 39 Context : p
Parse error - Unrecognized : }
--
mbstevens
http://www.mbstevens.com