please review my web hosting site, i have recently revamped the design so it is much easier to get around as the design is basic and clean.
the site is maginnonline.co.uk
Full Version: please review my web hosting site
There are many HTML syntax errors: http://htmlhelp.com/cgi-bin/validate.cgi?u...&spider=yes
The external style sheets seem very bloated. There's also some CSS between the HEAD and BODY sections (it should be in HEAD), but since it's inside Conditional Comments the validator ignores it.
You don't need to put every single list item inside its own list:
--in fact that defeats the purpose of lists.
The following seems very strange and unnecessary, not to mention it appears to result in invalid markup for IE6 (a table inside a link?):
Not sure if this is valid CC syntax either (the last ">" seems redundant):
The semantic structure of the HTML can be improved:
* Why put the text "Links" in a H1 element?
* Why use a definition list for the menu? An unordered list seems a better choice.
* Why not use H2 elements for sub headers (styled with CSS if necessary) instead of
* Why the BR elements between paragraphs?
You could also remove the fixed width of the layout, it shouldn't be too hard to make the content column resize to fit the user's window size.
The menu links "Home" and "Services" appear in two places, which seems redundant. At the same time "Services" and "Projects" go to the same page, which might confuse; ditto for "Link to us" and "Contact". Not sure what the link "Mobile" is about?
The forum link is broken.
http://www.maginnonline.co.uk/services.html uses inline frames, which is not recommended.
The external style sheets seem very bloated. There's also some CSS between the HEAD and BODY sections (it should be in HEAD), but since it's inside Conditional Comments the validator ignores it.
You don't need to put every single list item inside its own list:
CODE
<!-- Navigation item -->
<ul>
<li><a href="index.html">Home</a></li>
</ul>
<!-- Navigation item -->
<ul>
<li><a href="services.html">Services</a></li>
</ul>
<ul>
<li><a href="index.html">Home</a></li>
</ul>
<!-- Navigation item -->
<ul>
<li><a href="services.html">Services</a></li>
</ul>
--in fact that defeats the purpose of lists.
The following seems very strange and unnecessary, not to mention it appears to result in invalid markup for IE6 (a table inside a link?):
CODE
<li><a href="#">Clients<!--[if IE 7]><!--></a><!--<![endif]-->
<!--[if lte IE 6]><table><tr><td><![endif]-->
<ul>
<li><a href="login.html">Login</a></li>
<li><a href="livechat.html">Live Chat</a></li>
</ul>
<!--[if lte IE 6]></td></tr></table></a><![endif]-->
</li>
<!--[if lte IE 6]><table><tr><td><![endif]-->
<ul>
<li><a href="login.html">Login</a></li>
<li><a href="livechat.html">Live Chat</a></li>
</ul>
<!--[if lte IE 6]></td></tr></table></a><![endif]-->
</li>
Not sure if this is valid CC syntax either (the last ">" seems redundant):
CODE
<!--[if IE 7]><!-->
The semantic structure of the HTML can be improved:
* Why put the text "Links" in a H1 element?
* Why use a definition list for the menu? An unordered list seems a better choice.
* Why not use H2 elements for sub headers (styled with CSS if necessary) instead of
CODE
<p><b>About Maginn Online UK.</b></p>
* Why the BR elements between paragraphs?
You could also remove the fixed width of the layout, it shouldn't be too hard to make the content column resize to fit the user's window size.
The menu links "Home" and "Services" appear in two places, which seems redundant. At the same time "Services" and "Projects" go to the same page, which might confuse; ditto for "Link to us" and "Contact". Not sure what the link "Mobile" is about?
The forum link is broken. http://www.maginnonline.co.uk/services.html uses inline frames, which is not recommended.
I agree with the comments above about the fixed column width. The page doesn't fit on a 800x600 screen without scrolling sideways.
Bravo for allowing the user to increase/decrease font size.
I think most users still interpret underlined text as a link (as on the Services page). A color change or all caps might be a better way to emphasize text.
What is the link in the upper left-hand corner? Its title is "Go to Start Page", but it doesn't go anywhere.
The sentences in the first section of "About Maginn" should either each end in a period or should be a bulleted list with no periods. You also might want to have someone check other punctuation (submission solution's...home pc, or laptop...etc.).
Instead of the involved navigation tree of "Open a support ticket under sales here (submit a support ticket > Category > Sales)" under "How to Order", why not just link the user straight there?
I am not clear why you included the link to MySpace. If MySpace-type users are your target audience, verbiage to that effect should be on the site. But if this is a professional site for business users, I think the alluring (albeit attractive) photos of your MySpace visitors are an inappropriate inclusion.
The design is clean, neat, easy to navigate, and modern looking. But just a little color in the background to make the main column pop or for heading emphasis would enhance the site.
Bravo for allowing the user to increase/decrease font size.
I think most users still interpret underlined text as a link (as on the Services page). A color change or all caps might be a better way to emphasize text.
What is the link in the upper left-hand corner? Its title is "Go to Start Page", but it doesn't go anywhere.
The sentences in the first section of "About Maginn" should either each end in a period or should be a bulleted list with no periods. You also might want to have someone check other punctuation (submission solution's...home pc, or laptop...etc.).
Instead of the involved navigation tree of "Open a support ticket under sales here (submit a support ticket > Category > Sales)" under "How to Order", why not just link the user straight there?
I am not clear why you included the link to MySpace. If MySpace-type users are your target audience, verbiage to that effect should be on the site. But if this is a professional site for business users, I think the alluring (albeit attractive) photos of your MySpace visitors are an inappropriate inclusion.
The design is clean, neat, easy to navigate, and modern looking. But just a little color in the background to make the main column pop or for heading emphasis would enhance the site.
On the top left, is there some image that I don't see, or there's nothing there?
And, yes, little bit of colour wouldn't harm the site
And, yes, little bit of colour wouldn't harm the site
About 3/4 of the middle of the grey area above the top left HOME button has a "Go to Start Page" on hover...that does not link anywhere. That is what I was referring to.
QUOTE(non-profit @ Apr 17 2007, 10:56 AM) 
About 3/4 of the middle of the grey area above the top left HOME button has a "Go to Start Page" on hover...that does not link anywhere. That is what I was referring to.
Oh, I just looked. It is
<a class="sitelogo" href="#" title="Go to Start page"></a>
which is on the large top grey graphic.
# should probably be the url of the home page. Yes?
As I see things, http://www.maginnonline.co.uk/ is your homepage.
When clicking on "search sub" the menu will change which did really confuse me.
I would use some sort of sub menu if I was you.
I would use some sort of sub menu if I was you.
Well, it is very clean, and easy to navigate as well. I LOVE the "updates" on the left menu though it seems to me the date (i.e. "27/06/2007") should be of some other sort of text, perhaps bold, to distinguish one update from another.
This is a "lo-fi" version of our main content. To view the full version with more information, formatting and images, please click here.