Here’s a couple of small things.
The car image you have in this span should be in the background.
<div id="header">
<div class="header_text">
<h1 class="green">Lorem Ipsum Dolor</h1>
<p class="strong black">Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet.</p>
<p class="green">Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet.</p>
</div>
[B] <span class="header_car"></span>[/B] </div>
The span doesn’t sit nicely there either as it runs into a block element and therefore should be a block element itself. However you can apply that image to the header-text div anyway as it has no background.
<div id="header">
<div class="header_text">
<h1 class="green">Lorem Ipsum Dolor</h1>
<p class="strong black">Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet.</p>
<p class="green">Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet.</p>
</div>
</div>
#header {
min-width:994px;
position:relative;
overflow:hidden;
clear:both;
margin:0 auto;
height:222px;
background:#2f9f15 url(http://www.articlecon.com/demo/ed/temp2.1/images/header.jpg) 0 0 no-repeat;
min-width:40%;
}
.header_text {
position:relative;
background: url(http://www.articlecon.com/demo/ed/temp2.1/images/header_car.png) 100% 0 no-repeat;
height:222px;
}
I realise you may be adding other things there but you get the idea
[URL=“http://www.articlecon.com/demo/ed/temp2.1/images/header.jpg”]
This image is 97k and I just copied it into my paint package and it looks as though you can reduce it down to about 24k without much loss of detail at all. The image looks odd on my screen anyway as the background just finishes. You should make the right side of the image fade into a solid green color and then you can give the header background the same green colour so that the image doesn’t just appear to stop dead.
Some of the other images could do with optimising down also.
There’s an orphaned inline element here again.
</div>
<a class="samll_font" href="#">>> SEE ALL</a> </div>
Wrap it in p element.
</div>
<p><a class="samll_font" href="#">>> SEE ALL</a></p>
</div>
Much neater. (and what’s a samll_font ?)
Try not to use names like leftbar and rightbar and middlebar (I know I use names like that when I write demos but that’s just to make it easy for the demo).
Your middlebar would normally be something like #main or Content and the left and right columns could be could be called … #sidebar1 and #sidebar2 or #secondary. In that way it doesn’t matter if you decide later on that the left column should nw be on the right and you won’t have to change the names. You shouldn’t really identify the elements with a name that won’t make sense if they are moved somewhere else.
The only exceptions would be #header and #footer as you are unlikely to move the footer into the side column etc.
The round corners are a bit of a pain and make the markup a bit of a mess but whatever you do you still need the same number of hooks but I try not to nest everything and would usually do the top and bottom as separate elements rather than nested just to stop the html form being nested too deep. However there’s no real difference.
The reset could be trimmed and unless you envisage using all the elements mentioned you could trim that down.
You have a min-width set but it would be better if the page could go a little smaller as it is 1050px wide from the looks of it. Also perhaps think about a max-width as well to stop it getting too wide.
Hope that helps