treehouse : what would you like to learn today?
Web Design Web Development iOS Development

Website Evaluation

  • Hi Guys Ive been away for a while hope you are all well any chance of some feed back on a site Ive been working on? UK Microgen

  • Hi Polson,

    I'm watching the site in Google Chrome.

    First of I think the overal design is quite borring. Colors are to shiny, and the you could improve the photo style alot by hiring a photographer (Makes more trust of your company)

    • I think you should try another background - Its very bright.

    Additionally I think the design needs more whitespace - It seems very squeezed

    • And there's a problem here on the Quck contact form submit button

    Also I don't like the font-choise to much, maybe you could consider which is free to use.

    The Contact us page has a very green background, I don't think it fits the design - consider to remove this, keep it white, and maybe add a google map (What about contact information) ?

    Hope you can use this feedback,

    Have a great day,

    Regards Nils

  • I am not going to touch on the design, just code related stuff.

    All pages have the same meta description.

    A lot of pages have a h1 and h3 only (you skip h2).

    In the CSS, you have some issues with vendor prefixes, for example.

    You use " -moz-border-radius:4px;" and that's it. Border-radus doesn't need a prefix at all.

    On the contact page, in the custom CSS on that page, you use vendor prefixes for the box-shadow, those don't need prefixes either anymore and when you do use prefixes, you should always list the non prefixed version last to future-proof things.

    A good place to keep an eye on is http://html5please.com/ which will let you know what needs prefixes and what doesn't and also give you insight into fallbacks for important things, which neither a border-radius or box-shadow need. ;P

    You should get out of the habbit of typing "0px" and instead just use "0" in your CSS.

    The "For more information please feel free to call us today" how it has call us today in green looks like it should be a link, but is not, just a personal annoyance. This actually applies to a lot of the green text in some areas.

    Some of the class names, like "greenbold" are bad in that they should not be descriptive of the color, what about when the client wants pink... then it may feel dirty having pink text on a .greenbold class.

    You could also apply that green "check" as a psudeo element on list items or a span through CSS, instead of including it as an image, after all it is a styling element.

    On the main navbar, you should make the hit areas on the links bigger, usually this involves adding the padding to the link element, not the LI to provide a bigger click location.

    This site would also be a perfect candidate for WordPress, that would fix a bunch of dumb issues like having duplicate things like "http://www.ukmicrogen.co.uk/", "http://www.ukmicrogen.co.uk/index.html", "http://ukmicrogen.co.uk" and "http://ukmicrogen.co.uk/index.html" all resolving which is bad for various reasons.

    Just a few things I noticed quickly :P