Need some critique on a website

Discussion in 'HTML & Website Design' started by draconusit, Jul 16, 2007.

  1. bpb2221

    bpb2221 Peon

    Messages:
    83
    Likes Received:
    1
    Best Answers:
    0
    Trophy Points:
    0
    #21
    $599.00 for multiple professionals versus the hours/days/weeks you would spend laboring and still coming up with something that looks second rate is a bargain in my opinion. It is all about opportunity cost. Then again, you might be a graphic artist (I'm not), so it would be of great value to me.

    Ben Blanchard
     
    bpb2221, Jul 23, 2007 IP
  2. deathshadow

    deathshadow Acclaimed Member

    Messages:
    9,732
    Likes Received:
    1,999
    Best Answers:
    253
    Trophy Points:
    515
    #22
    Appearance aside, you went a little DIV and Class happy in your code.

    For example:
    <div id="navigation">
    <div class="navi" style="width: 105px; height: 32px">
    <a href="http://www.draconusit.net/news.html">News</a></div>
    <div class="navi" style="width: 105px; height: 32px">
    <a href="http://www.draconusit.net/tools.html">Tools</a></div>


    etc, etc, etc.

    They are all the same height and width so you should be declaring that in your CSS - that's a list of items, so it should be a UL/LI - you have a parent class wrapping all of them, so the child items don't need to have a class... and you should be serving that file from the same directory as the links point to, so declaring the http://www.draconisit.net should be unneccessary.

    For that entire section all you should need is:
    <ul id="navigation">
    	<li><a href="news.html">News</a></li>
    	<li><a href="tools.html">Tools</a></li>
    	<li><a href="services.html">Services</a></li>
    	<li><a href="Shop/index.php">Store</a></li>
    	<li><a href="forums/">Forums</a></li>
    	<li><a href="links.html">Links</a></li>
    	<li><a href="contact.html">Contact</a></li>
    	<li><a href="cgi-bin/chat/chat.cgi">Chat</a></li>
    	<li><a href="about.html">About Us</a></li>
    </ul>
    Code (markup):
    Improper use of headers, you've got a H2 up top where that should be a H1 - and following a header by a header for subtext is just wrong, especially when the subtext has no content. You've got excess DIV's around items coming out the yazoo...

    and I spy a depth sorting error on your bottom shadow... and you may want to use a side image the size you are using the image - as resized down like that it looks like garbage in FF.
     
    deathshadow, Jul 23, 2007 IP