View Single Post
  #6 (permalink)  
Old Nov 14th, 2007, 10:02
MikeHopley MikeHopley is offline
SuperMember

SuperMember
Join Date: May 2007
Location: UK
Age: 27
Posts: 1,111
Thanks: 0
Thanked 0 Times in 0 Posts
Re: Block Architecture

I like the simple, clean design.

Your text is too small. Your logo and navigation are in the wrong place: they should be at the top of the page, not the bottom. Why? Convention. Conventions make websites easy to use. On 800*600 resolution, none of your navigation is visible without scrolling.

In particular, the navigation is very hard to spot. I thought at first that your site had only one page!

Well done for writing valid, Strict code. It's a pleasure to see someone using <dl>s as a title/content pairing method; but in this example, surely the items are structural headings. Here's the structure you should have:
Code: Select all
<h1>Block Architecture</h1>

<h2>News</h2>

<h3>October 2007</h3>

<p>Block Architecture were shortlisted...</p>
<dd>s should not be used as a replacement for paragraphs. Inside a <dd>, you may include <p> elements. And this code is just terrible:

Code: Select all
<dd>&nbsp;</dd>
This is a novel variation of the usual "<p>&nbsp;</p>" rubbish. Use CSS margins or padding instead.

Last edited by MikeHopley; Nov 14th, 2007 at 10:12.