New Site for Review...
newbie
Posted: Sun, 2008-07-13 20:45
Hi Guys,
OK I'm new here and would like you CSS design masters to check out a site a have nearly completed for a company in my area and let me know what you think. IT's all done bar the Free Response Form on the right which needs setting up to e-mail the owner. Any any comments would be great. Thanks.
Regards
Gareth
Regular
Posts: 34
Joined: 2008-07-10
Location: UK
HTML CODE
Posted: Mon, 2008-07-14 01:08
I think your site and the CSS is great.
The design isn't to my taste, but it looks good enough to me.
The only thing I would change would be the contact method.
I don't use software for email and the contact tab forces me to open outlook. Would be better to have a page with a simple PHP contact form allowing you to add the address details and a link for email should customers choose to contact you that way.
Other than that its great, well done.
id="css"
#css {
background:too tired for this!
}
newbie
Posts: 5
Joined: 2008-07-13
HTML CODE
Posted: Mon, 2008-07-14 09:31
Hi JCK,
Thanks for the comments and suggestions. Yes you are right I am going to be setting up a PHP contact form, that link is just their until it is done but thanks for suggesting anyway.
What is it about the design you don't like, or is not to your taste? Do you have any pointers at all?
Thanks again.
Regards
Gareth
Enthusiast
Posts: 208
Joined: 2008-03-16
Basically nice
Posted: Mon, 2008-07-14 09:58
Well, it is basically nice, not really a knock out design but a good start. It lacks some spice. But artisticly handicapped myself I can't blame you.
To my feel the header is missing something, looks empty and amateurish.
However, there are some things that I saw that could easily be optimized. But first of all, I like the default rather large font-size and that you work with em on all fonts.
1) Your headings are h1 for the logo, which is good, but the h2 is your response form, and h3 is only for the actual content. I would either do it vice versa or use h2 on both, since they are different sections of your page.
2) The contents headings are too small. They are hard to see, and the blockquote beneath them is nearly double in size. I would not even use an image
3) Your navigation is too small, in font and in tab size. Increasing the font size through the browser breaks the layout.
4) Your form is out of line, label and inputs are not on the same height (at least in FF3 and IE6). You are using an ol for the form. First of all, it should be a ul, second you shouldn't use any list. Your form is so basic that you can achieve a better result by simply using the form elements right. Using margins on the elements saves you the ugly br tag, and you styled your ol anyway.
5) The pixels you save doing so can be used for acessibility, for example using a legend and the for attribute on the label and an id on the input, or simply wrapping the input into the label. There are few real good sites that show a form, one of them is The man in blue.
Besides, why did you put your form in a blockquote???
6) Finally, your CSS indicates a 100% width, but it isn't when I decrease resolution. You might want to check why that is since I didn't see a min-width.
7) I find you are using too many hacks for this simple layout anyway
Leader
Posts: 605
Joined: 2005-01-17
Location: Pittsburgh
Label issues
Posted: Mon, 2008-08-11 04:43
You're not using labels correctly. In your right column, the labels, "Title", "Forename", "Surname," etc., are not pointing to anything. You have:
HTML labels should point to a form element with an ID. The value of the 'for' attribute in the label must match this form element ID. You should have:
Clicking on the label should focus the cursor on the form element it's pointing to.