View Full Version : Review my first paid website
08-23-2009, 03:28 PM
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
08-23-2009, 07:20 PM
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.
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.
08-23-2009, 07:39 PM
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.
08-23-2009, 07:46 PM
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.
08-23-2009, 09:31 PM
Oh I see. Thanks again for the help.
Powered by vBulletin® Version 4.2.2 Copyright © 2017 vBulletin Solutions, Inc. All rights reserved.