The guys seem to have picked the bones out of this one, but I'll add a couple of points...
Give your text a little less line height - it looks almost double spaced at the moment, and that's too much. You should also add a little padding to the left & right of your areas of text - they look really cramped when the text butts right up to the border. I'm not wild about the green colour either - I'd keep it for headings but switch to black or very dark grey for the body text.
You've got a serif font in the title bar (is it me, or is the "PTY" bit in a slightly different font to everything else?), sans-serif for the menu, serif again for the content and sans-serif for the footer. That's too many switches of font for my money. Serif fonts can look classy in a title or logo, so I wouldn't necessarily change that too much, but I think you should pick one font for the rest of the site. Personally, I favour sans-serif for technology businesses, but if they're after the "traditional" look, go with serif.
Under the hood, I've seen a lot worse. It was a pleasant surprise not to see any table markup there, so kudos for that.
There is acres of Javascript at the top of each page. For some reason you've got your photoshuffler script repeated twice, on the pages that don't use it at all! I suggest to wrap up all your bits & pieces of JS into an external file and just load it into your pages like this:
Code:
<script type="text/javascript" src="/brocks.js"></script>
It'll help your pages load quicker and also no risk confusing any validators.
Use the <p> element for paragraphs. You're doing this on most of the pages I looked at, but the home page has <br>s instead.
Resist the temptation to use inline CSS in your pages - it helps maintain discipline over keeping a consistent look and feel across pages, and it makes it easier to tweak things globally as you fine-tune the look.
You could consider marking up instances of the company name like this:
Code:
<span class="coname">Brock & Associates</span> is a bonzer company.
You can then decide globally how (if at all) to highlight it. Your current look would be
Code:
.coname {
font-weight: bold;
font-style: italic;
text-transform: capitalize;
}
Personally, I think bold, italic
and all caps is a bit too much - but that's why I'd mark it up with easy tweaking in mind!
Despite all the above, it's very impressive for a first attempt.
-- Chris Hunt
Webmaster & Tragedian
Extra Connections Ltd