Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Results 1 to 8 of 8

Thread: Site Review

  1. #1
    Regular Coder
    Join Date
    Jul 2002
    Posts
    698
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Site Review

    I got this page

    www.the-hive.dk/~donp/

    I would like if some would look at it and give back some constructive criticism

  • #2
    Senior Coder
    Join Date
    Aug 2002
    Location
    A 4D universe
    Posts
    1,337
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Firstly, your page is somewhat broken in Opera (v7.23). The menu category bars extend out of the menu region.

    Second, try and introduce some padding, it is a wise idea, seperates the text from its borders.

    Third, it is nice to see that you have used a tableless layout, however, your code does not validate @ HTML 4.01.

    Fourth, you state font-size as a fixed size, in this case, .pt, it isn't wise, use %'s instead. That will allow people who use IE to increase the font size. Your page should be flexible enough to allow that.

    Last, the layout/colour scheme. It seems a little packed into a small place. But that isn't very important, it is the colours/font sizes that you use that cause it to look cramped.

    The colours do not compliment each other, and clash with the background colour that you have used. Try using a dark shade of brown instead of the navy blue. Further, there is no way of telling what is a link, well not really. Try and introduce hover links. It's more user friendly.

    That is about all I can think of, anything else is minor, jibber jabber.


    http://www.mudsplat.com - Web design, print, and marketing solutions.

  • #3
    New Coder
    Join Date
    Jul 2003
    Posts
    88
    Thanks
    0
    Thanked 0 Times in 0 Posts
    It isn't very visually interesting. Try adding something to catch attention, and then let it go. ala: http://evolt.org/article/rating/22/60261/index.html

    Also read up on some colour theory, because your colours just don't go together. See: http://www.worqx.com/color/index.htm

    In both of the above links you will find the theories behind good visual design. It is up to you to actually use them

    Later,
    Douglas
    it's web design. me me me them everyone.
    ~ No more Netscape. No more IE. What is the world comming to? ~

    __ :: Our Tiny Planet

  • #4
    Regular Coder
    Join Date
    May 2003
    Location
    Manassas, VA, USA
    Posts
    220
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Actually, it's best to use em's instead of %'s (btw, ionsurge, i noticed that was your 1234th post )

    As for the design, I would suggest going with a different (darker) blue for headings. It's to bright compared to the "mustard" yellow of the heading background. I also mught suggest changing the white borders to black or something darker, as they don't contrast enough.

    You might also want to change the font to a sans-serif font like Verdana (Times just doesn't look right to me.)

    -Andrew
    Get the best Web Design books called Visual QuickStart Guides at the Peachpit Press
    Ithium Web Hosting

  • #5
    New Coder
    Join Date
    Jul 2003
    Posts
    88
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by STDestiny
    Actually, it's best to use em's instead of %'s
    Best to use both:

    Code:
    body { font-size: 80%; font-family: .... }
    h1 { font-size: 1.2em; }
    p { font-size: 1em; }
    general font information...
    http://thenoodleincident.com/tutoria...ont/index.html
    and concludes with...
    http://www.thenoodleincident.com/tut...phy/index.html

    But anyway...

    Douglas
    it's web design. me me me them everyone.
    ~ No more Netscape. No more IE. What is the world comming to? ~

    __ :: Our Tiny Planet

  • #6
    Senior Coder
    Join Date
    Jun 2002
    Location
    The Netherlands, Baarn, Ut.
    Posts
    4,253
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Details, really

    Layout is straightforward, there's not too much visual interest, but that's fine by me.
    I would recommend having the header as seen on the home page return on the other pages as well to create some sort of unity.
    Codingwise, everything looks pretty tidy; if you look into possible validation errors I'm confident that you can get it XHTML 1.0 strict. not necessary, but aim for excellence, right?
    I would however like to point to the over-use of span elements; things like:
    Code:
    <span class="Headline">
    can easily be replaced by:
    Code:
    <h1>
    and styled to taste; users with non-graphical browsers will like it, and so will search engines. Unless you don't want to be found, of course...
    Regards,
    Ronald.
    ronaldvanderwijden.com

  • #7
    Senior Coder
    Join Date
    Aug 2002
    Location
    A 4D universe
    Posts
    1,337
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by DougBTX
    Best to use both
    Using both, that is wrong. It simply eradicates the purpose of what I said, the browser will simply use the latter statement, e.g. if it was a fixed size, e.g. 14px, the browser would use that, rather than the previously stated % or vice versa.

    You only need to use one statement, unless you are using different size values. Remember you can use values above 100%, e.g. 150% for larger text.
    http://www.mudsplat.com - Web design, print, and marketing solutions.

  • #8
    Regular Coder
    Join Date
    Jul 2002
    Posts
    698
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Changed the blue to brown.
    made some padding.

    gonna change to h1,2,3 tags as suggested they fill less bytes, I was under the impression though that a document where only allowed to have 1 of each.

    as for not being able to distinguish links from the rest of the stuff I am a bit confused seing as all links are colored Teal.

    and for not validating at the 'oh so migthy' w3c doen't really matter that much to me, the fault anyway is in how I write <ul> and <li> tags


  •  

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •