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 5 of 5
  1. #1
    New Coder
    Join Date
    Jul 2009
    Posts
    52
    Thanks
    6
    Thanked 1 Time in 1 Post

    Review my first paid website

    Hey just wanted to get some reviews on colors and the overall general layout plus if the typography is any good on the landing page.

    The url is http://biggtire.com

  • #2
    Supreme Master coder! _Aerospace_Eng_'s Avatar
    Join Date
    Dec 2004
    Location
    In a place far, far away...
    Posts
    19,291
    Thanks
    2
    Thanked 1,043 Times in 1,019 Posts
    It looks like a christmas theme with that alternating color bar going across the top. Something about it makes it look outdated, like the site name in a repeating background image. There is about 100px of horizontal scrollbar in Firefox 3.5.2 on my 1280x800 resolution.

    I'm pretty sure you aren't allowed to use the Michelin mascot for the site unless you some how got permission to do so.

    It just looks a bit "amateurish". I've never really liked sites that put their url in the site header. We know what site it is, no need to tell us again.

    And the code is another thing. You are using tables for page layout. Tables are NOT meant for page layout. Read the link in my sig titled "Why Tables for Layout is Stupid?". You also have some coding errors.

    http://validator.w3.org/check?uri=ht...Inline&group=0

    These are errors that should not exist especially if you are getting paid to make this site.

    Block level elements can't go inside of inline elements like spans. So no H4 in a span.

    I just checked your contact form. I like that you replaced the checkboxes with something a little more attractive however if the user has JS disabled those checkboxes become useless as they don't degrade by to browser defaults. I also suggest giving the user a calendar of some kind for the date box so they can visually see the day and date.
    Last edited by _Aerospace_Eng_; 08-23-2009 at 07:25 PM.
    ||||If you are getting paid to do a job, don't ask for help on it!||||

  • #3
    New Coder
    Join Date
    Jul 2009
    Posts
    52
    Thanks
    6
    Thanked 1 Time in 1 Post
    your right i thought it looked a little christmassy too which my client said he loved i guess ill try to talk him out of it. I don't know what else should i put in the header also my client does have permission to use the michelin man.. I used tables for tabular data in the content box which is how there suppose to be used (correct me if i'm wrong) i used divs for the actual layout. But pretty as this is my like third site I have ever designed it is decent and mostly everything else you said about the code i didn't know so I'll work on fixing what you didn't and what you said was wrong. Other than that thanks for the input.

    EDIT: Just read Why Tables for Layout is Stupid? and your right ill review the site a little more and redesign some parts of it.
    Last edited by firetag; 08-23-2009 at 07:50 PM.

  • #4
    Supreme Master coder! _Aerospace_Eng_'s Avatar
    Join Date
    Dec 2004
    Location
    In a place far, far away...
    Posts
    19,291
    Thanks
    2
    Thanked 1,043 Times in 1,019 Posts
    The places you are using tables on isn't data. Its content. Data would be something like parts listings that has details about the parts. Data would be your email inbox with a list of emails.
    ||||If you are getting paid to do a job, don't ask for help on it!||||

  • #5
    New Coder
    Join Date
    Jul 2009
    Posts
    52
    Thanks
    6
    Thanked 1 Time in 1 Post
    Oh I see. Thanks again for the help.


  •  

    Posting Permissions

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