Web design & SEO site for review

OK, you ready?

Not bad, but could be better. I'm not talking about design cause I guess I'm a form-follows-function person. I'd rather a site were butt-ugly and worked beautifully than other way around.

Visually there's something going on with the little thin grey border right under the menu... see the sides. This is in FF3 (I didn't check in other browsers as I'm not at my machine).

I've never actually seen generated CSS from this CSSCreator, Lawlz. I take issue with this:

#onecol a:link { color: #336699; text-decoration: underline;}
#maincol a:link { color: #336699; text-decoration: underline;}
#leftcol a:link { color: #336699; text-decoration: underline;}
#footer a:link { color: #336699; text-decoration: underline;}
#copyright a:link { color: #336699; text-decoration: underline;}

Assuming each container really did have to get mentioned (didn't look to see why):
#onecol a, #maincol a, #leftcol a, #footer a, #copyright a {
color: #369;
}
#onecol a:focus, #onecol a:hover, #maincol a:focus, #maincol a:hover, #leftcol a:focus, #leftcol a:hover, #footer a:focus, #footer a:hover, #copyright a:focus, #copyright a:hover {
text-decoration: none;
}

Or more likely,
a {
color: #369;
}
a:focus, a:hover {
text-decoration: none;
}

I didn't look at the rest of the css.

I looked at your front page with images on and off, and CSS on and off.

Because the text at the top is important, the name of your site, I would actually probably either use image replacement (so with images off but css on there's some nice, maybe white, readable text) or try adding text styles to the image (not sure if that can style the alt text).

Your bullet points on the right really look like lists. I would mark them up as such, even if that's more likely to cause a bit of grief re IE. If I were a JAWS user I would really appreciate that, being able to skip from list to list with more ease than a bunch of p's who aren't actually paragraphs.

You are suffering from classitis:

etc...
      <p class="serind"> Domain Name &amp; Web Hosting Advice </p>
      <h3 class="serhead"> Search Engine Marketing: </h3>
      <p class="serind"> Search Engine Optimisation (SEO)</p>
      <p class="serind"> Natural Link Building Strategies</p>
      <p class="serind"> Search Engine Marketing (SEM)</p>
      <p class="serind"> Pay Per Click (PPC) Management &amp; Training</p>
      <p class="serind"> Affiliate Marketing Advice</p>
      <h3 class="serhead"> Real World Marketing Strategies: </h3>
      <p class="serind"> Establishing Routes to Market </p>
      <p class="serind"> Public Relations (PR)</p>
      <p class="serind"> Email Marketing Campaigns &amp; Newsletters</p>
      <p class="serind"> Sales Lead Generation </p>
etc...

They say, every sperm doesn't need a name, only the one who's special. Every single p is class="serind" and every single h3 is class="serhead". I'd simply style the p and h3, or even #maincol p and #maincol h3. Otherwise you're kinda doing the George Foreman thing-- didn't he name ALL of his kids "George"? Might as well just not name them at all and call them "George's kids." Means the same thing.

I'd swap the order of Maincol and leftcol. The names suggest they are in the correct order, but your h1 (the Name of the Page) is near the bottom. Since they're sitting next to each other they could have their float directions switched.

Be careful with two 50% width divs side-by-side-- IE6 has been known to employ what I call "Redmond Math"-- where 50% + 50% = 101%. I don't have IE6 on hubby's machine here but would expect float drop-- esp since the page container has a set px width. To be safe I'd have like 49.9% + 49.9% instead.

On some of your links you have the famous "read more" links... yes, we're all guilty of those, but they are considered bad form both for those using linear browsers (like via a screen reader who can skip from link to link) and for those who are simply to lazy to be bothered reading things like "context" (most users, unfortunatley... see how many people don't even read instructions on highly dangerous things). You have excellent links with the contact form (twice), the portfolio image (alt text), and the terms&conditions-- that's the way to link, if you're really thinking about going at it all 100% and perfection and whatnot.

As a user I expected to be able to click on the snap images for a closer look (watermark them or something). Esp since that's under "graphic design" and looks design-y while the "web developement" looks... more like something I'd build, lawlz. Functional. So let people get a better look at your graphics side (or add it to your own personal site to let people see it and touch it first hand).

Your form page:
notes:

<script type="text/javascript" src="http://d807497.cm43.creativeministries.co.uk/scripts.js" <span style="font-weight:bold">language="javascript"</span>></script>
<style type="text/css">
<span style="font-weight:bold"><!--</span>

language=Javascript needs to go, as does the HTML commenting from the CSS. If I recall correctly it was Nutscrape who started that language=javascript stuff. It's not even allowed in either HTML4 strict or XHTML, I forget which. It's not needed for either.

onSubmit="javascript: return checkform();"

I am huge against relying on the client to validate forms. Not everyone has JS (and you may have specifically used this because it's often spam bots who don't have JS); form validation (and processing) should ALWAYS happen on the server side, unless you are serving an intranet where you have total control over the user agents and the folks using them. (rubs hands together, pets cat, evil laugh... muhahahah, BOFH kinda thing)

<select name="title" size="1" class="txt" id="title"<span style="font-weight:bold"> select</span>>

          <label for="name"> Name*: </label>
          <input type="text" name="name" id="name" class="txt" />

Good on using the for attribute in the labels... hell, even actually using labels. Too many forms don't even use proper form controls.

Those hidden inputs should throw an error-- they are direct inline children of the form, which forms may not have.

Wrapping the inside of the whole form in a fieldset (conveniently a block element, lawlz) would be a good thing, though its use does pretty much require a legend (though the validator may not explicitly say so when you have divs in the form... I know, I use divs for the same reason in forms and I've checked my error messages... it was either HTML4 or XHTML who acted like it wasn't required for a fieldset to have a legend, but don't be fooled!).

Your final list of links (in the footer) should also be in a list. Even though the way you have it now is less code.

And this:

  <p>&nbsp; </p>

Overall, I like it, though the design is a bit sparse (which I tend to like). I'd possibly rearrange the h1/2/3 order for header-tabbing (in JAWS for instance) and SEO reasons (though you may have specifically chosen it the way it is for SEO reasons) and I tend to expect the h1 near the beginning of the document..

While you mention the "much misunderstood SEO" stuff, I'd encourage you to go ahead and add a little education for your readers/customers. A few hints on the site (that you know how things like headers and clean, semantic code affect the "meaning" and therefore the ranking of page content) points to a person knowledgable in SEO instead of simpply saying "I totally rawk at SEO!" : )

Good luck with your business!

Thanks for your comments Stomme poes will have to spend some time digesting them and get back to work...

I really appreciate the time that this must have taken.

Justin

Well justin your website is really awesome and very SEO friendly as well i barely found any mistakes on the website.Keep it up...you had made and excellent website

SEO course karachi

mod edit: Spam link removed. Wait until you've done due diligence, and earned a sig. This is your second empty post with a spam link. More will get your account blocked. --gary.turner

You are totally right, he's crushing your stats in SEO. Also, dude seriously, TABLES all over your site, dude seriously.