New Site

Followers

August 26, 200818 yr

I'm creating this site for the company i work for, please let me know what you think.

a couple things first off...

i know it don't look right in safari ( i have no love for apple)

i know it don't work in IE6 i may get round to fixing this otherwise i have no love for you if your still using it just upgrade!!

i know the form's dont work and i know alot of the images are used alot as i havn't got images to replace them with yet.

thanks

Visit My Website

August 27, 200818 yr

What strikes me more than anything is that its a bit boring with the overuse of dull colours like gray.

Also the images in the meet the team section scare me.

That aside its not a bad start.

The site seems to have a lot of content and might lend its self to a gridview style.

http://www.markboulton.co.uk/journal/comme...systems_part_4/

Maybe the above will help you.

August 27, 200818 yr

Here's a harsh, but hopefully helpful, critic.

First of all saying you don't care about a massive user base is arrogant and WILL loose a lot of traffic/business. Get it sorted no excuses (I'm sure the company you done it for would also want it done).

Then, 43 validation errors...come on you can do better than that!

Too much space at the top of the site (grey bit), I want to see grey space because?

Navigation, some sort of highlighting would help identify where the mouse is.

Grey is a little overused/overpowering in general.

Maybe adjust your text line height and spacing, it looks a little cramped.

(How is the site copy-writed, I hate seeing them if meaningless)

Right onto the code:

Style sheets in the body!

And then elements are styled in-line!

You've got about 100 div containers before the main content, what the.....

I'm going to stop there it's too messy. It's obvious that you're used to using tables as you've just replaced a <table> with a <div> - good for you for the change-over but divs are so much cleaner, you don't need to contain your code so much...eg one container for an image!!?

It's not an awful site, but it needs a lot of polish in the design and the code is really confusing and messy.

August 28, 200817 yr

Author

thanks for your comments i have used them to improve a few things.

Firstly - thanks for saying about there being too much gray thats what i said myself, but the company like it like that, so not alot i could do, i have though made the background a darker gray.

secondly - ive reduced the gap between the content and the top of the browser.

thirdly - i added a mouse over effect on the menu as suggested, now the link go italic when you mouse over.

fourth - sorted out MOST of the validation errors just stupid little things really, but there is still 3 errors, and i dont quite get what they mean? could someone help me out on these?

Line 1, Column 120: DTD did not contain element declaration for document type name.
…R/xhtml1/DTD/xhtml1-transitional.dtd">✉

A DOCTYPE declares the version of the language used, as well as what the root (top) element of your document will be. For example, if the top element of your document is <html>, the DOCTYPE declaration will look like: "<!DOCTYPE html".

In most cases, it is safer not to type or edit the DOCTYPE declaration at all, and preferable to let a tool include it, or copy and paste it from a trusted list of DTDs.

Line 2, Column 42: document type does not allow element "html" here.

<html xmlns="http://www.w3.org/1999/xhtml"><head>✉

The element named above was found in a context where it is not allowed. This could mean that you have incorrectly nested elements -- such as a "style" element in the "body" section instead of inside "head" -- or two elements that overlap (which is not allowed).

One common cause for this error is the use of XHTML syntax in HTML documents. Due to HTML's rules of implicitly closed elements, this error can create cascading effects. For instance, using XHTML's "self-closing" tags for "meta" and "link" in the "head" section of a HTML document may cause the parser to infer the end of the "head" section and the beginning of the "body" section (where "link" and "meta" are not allowed; hence the reported error).

Line 168, Column 14: no document element.

</body></html>

cheers

Style sheets in the body!
And then elements are styled in-line!

You've got about 100 div containers before the main content, what the.....

I'm going to stop there it's too messy. It's obvious that you're used to using tables as you've just replaced a <table> with a <div> - good for you for the change-over but divs are so much cleaner, you don't need to contain your code so much...eg one container for an image!!?

wheres style sheets in the body :S

and as it happens im not use to using tables at all :S never used the jumped straight in with DIVs and CSS but im abit rusty havn't done it for a while, the company employed me in a different role and just aked me to do the site aswell.

thanks for your comments and help