Free Template: Peephole

[Neoboffin]

Preview
Download

Suggestions and comments are always welcome. :o]

[Magnus]

You're using way too many divs. Also your headings are wrong.

<h1> does not belong semantically to "take a pick" it belongs to the "peephole" logo up at the top. I suggest putting that as your first heading (<h1>) Change it from an <img> to a <div> and set the image as the background. Insert the text <h1>PeepHole</h1> inside the div, and set

CODE

text-indent: 2000px;

) Its a useful technique. that way if you remove all the styling the text logo is still there (helpful for people using screenreaders and what not). Apple uses it on their site.

as for your divs, you don't need a #wrap div, body can be you wrapper. you also don't need a div just to clear the two columns. Your footer div can do that.

As for #middle and probably #top you're just using images. You don't need divs there either.