Novice Seeks Critique(s)

my first time helping, hope i get this right…

looks cool. but the other guys are gonna get on you about validity … in Firefox, you seem to have some text overlapping your links lists. maybe it’s a table issue, i don’t know. if it is, the earth will blow up.

i love wine. ..or maybe I just like to drink. Either way. nice stuff.

to dispell that overlapping text issue, in your sidebar i would just follow the format of the #menu div for the list items (li) in the index.php.

and in your wp-layout.css create a span tag like :

.grape {
color:whateverTHEhexadecimalFORpurpleIS;
font-size:x-small;
}'</code>
so from within the index.php where you wanted the text to be grape, you'd wrap 'em in "span" tags.
<code><span class="grape">Traditional Winemaking, Contemporary Wines, Extraordinary Service.</span>
'

and instead of that table for the footer, I would create a div called #footer in your wp-layout.css and play with the margin and padding attributes until I got what I wanted.

Hope that helps. Time to drink.

Your page gets a little [read: a lot] wide at high monitor resolutions. Not a huge problem but the lines of text in your content [at these resolutions] get a big long.

this may sound a lil harsh but if i don’t criticize it assume i like it ^_^

1) i agree with RSL on this one. i use 1920×1200 and the page looks very very wide. use a fixed width layout if possible — something around 760px shouldn’t cause too many issues.

2) your sidebar looks a little funky since the bullets all show. you may want to remove the higher list’s bullets and change the bullet type for the lowerlevel lists.

3) the font-sizes don’t fit very well. i think the problem is that your sidebar text is too small for the rest of the page. if you fixed width, you may need to shrink the rest of the text to that.

4) validation. gotta try to fix that :P.

edit: just looked at your code. try to lose the tables in your layout too (except the calendar, of course).