Please critique this site

It's a site for our local (Dutch) tennisclub. I've tried to use css
and w3c correctly.

Link: http://www.ltvborgharen.nl

I've found in Mozilla and Opera that the thin black line surrounding
the site doesn't show (in IE it does).

All comments are welcome

Dan

ddanielrijs (AT) hotmail (DOT) com wrote:

It's a site for our local (Dutch) tennisclub. I've tried to use css
and w3c correctly.

Link: http://www.ltvborgharen.nl

I've found in Mozilla and Opera that the thin black line surrounding
the site doesn't show (in IE it does).

The design looks good to me. I'm not in favor of fixed-width pages, and
if you intend to add more content, you should look into that.
http://allmyfaqs.net/faq.pl?AnySizeDesign

Consider using better class names in the CSS. p.green will look silly
if you later decide to change that class to brown. I usually use words
like .. "p.cool" .. "p.hot" .. "p.veryhot" :-) Not critical, though.

There is one error each in the HTML and the CSS:
<http://validator.w3.org/check?verbose=1&uri=http%3A//www.ltvborgharen.nl/>
<http://jigsaw.w3.org/css-validator/validator?profile=css2&warning=2&uri=http%3A//www.ltvborgharen.nl/>

--
-bts
-Warning: I brake for lawn deer

On Sun, 12 Mar 2006 20:47:03 +0100, ddanielrijs (AT) hotmail (DOT) com wrote:

It's a site for our local (Dutch) tennisclub. I've tried to use css
and w3c correctly.

Link: http://www.ltvborgharen.nl

I've found in Mozilla and Opera that the thin black line surrounding
the site doesn't show (in IE it does).

All comments are welcome

Dan

Look what happens with your menu when you turn images off.

You're right, find no trace of the thin black line.

--
o'tom po'tom

ddanielrijs (AT) hotmail (DOT) com wrote:

Link: http://www.ltvborgharen.nl

I've found in Mozilla and Opera that the thin black line surrounding
the site doesn't show (in IE it does).

Fix the HTML validation error.

The a:hover color has very poor contrast. Dark rather than light gray
would be better. And the hover completely disappears when images are disabled.
Since you have a mouseover for body links, why not use that also for the
navigation links? Oh, I see why: a <map>.
The bullet image for the News list would look better if its background
were transparent.

The thin black border is not exactly missing: the bottom portion is
under the top image.
You float #middleblock which contains everything else. And leaves no
normal flow content. So the #MainContent div wraps what is normal which is
only the header. Remove float:left from #middleblock and the div extends
to include it as well giving you the black border as you expect it.
IE incorrectly wraps the floated block.
See <http://www.positioniseverything.net/easyclearing.html> and
<http://css.maxdesign.com.au/> for more info on floating.

--
jmm (hyphen) list (at) sohnen-moe (dot) com
(Remove .AXSPAMGN for email)