| mbstevens 2006-06-08, 7:28 pm |
| Chaddy2222 wrote:
> Hi guys, I just finished a re-design of http://www.freewebdesign.cjb.cc
> and I would appreciate your thaughts.
> It is produced with Valid XHTML 1.0 Strict and valid CSS.
> Your feedback is much appreciated.
You have a useless home link on the home page, and every following page
has a link to itself.
It is OK to have a couple of " " characters between links, but you
should also have a printing character, like '|' to separate them.
You have two "Contact Us" links on the home page. This is not good in
itself, but the second one also is styled like a button link and looks
out of place in the middle of a paragraph.
Learn to use indentation to structure your markup -- it will make it
much easier to maintain, and lead to fewer errors. It would certainly
have made my reading of your markup more pleasant.
You should really be using styled paragraphs instead of all those "<br>"
and " <br>" combinations.
All those are minor, however -- you're validating up nicely in both HTML
and CSS. And it looks good. The main thing is to be a bit more logical
about your links.
--
mbstevens
http://www.mbstevens.com/
|