Mon, 2010-02-15 02:09
Please check out my latest web project. I'd love to hear what everyone thinks about it so far, from the design to the dev.
Thanks!
Curtis
Mon, 2010-02-15 02:11
#1
purewebdesigner wrote: Please
Please check out my latest web project. I'd love to hear what everyone thinks about it so far, from the design to the dev.
Thanks!
Curtis
For some reason I can't seem to edit my post.
Website is - http://www.puredezigner.com/clients/jbpub/
Mon, 2010-02-15 11:41
#2
Looks good very
Looks good very rich/sumptuous which is good for a site to do with food/accommodation not sure I like the links on dining page hovering to a darker colour; tend to loose all the text in the background.
N.b Haven't looked underneath the hood (American for the proper term bonnet)
Mon, 2010-02-15 13:33
#3
Hugo wrote: ... Haven't
... Haven't looked underneath the hood (American for the proper term bonnet)
That's because America has tough and manly cars and tough cars don't wear bonnets. 
PWD, I'm very jealous of your design skills. Looks great. Are you local to that place?
Mon, 2010-02-15 17:51
#4
This site design was actually
This site design was actually created by Kiska Media I help them with development from time to time. Wish my skills were this good too. 
I'm not to far from there, I'm located right outside of DC.
@Hugo I agree with you about the text color. The up and hover state colors were reverse but this was to hard to read I think we need lighter colors for these. I'll run this by the designer.
Thanks for your feedback.
Tue, 2010-02-16 08:13
#5
I would replace the "brown"
I would replace the "brown" text to another color.
Wed, 2010-02-17 03:17
#6
You've got title and alt
- You've got title and alt attributes on your images; images don't need title attributes if an alt is in place. You could put a title on the anchor around the images, but provided the alt attribute sufficiently describes the image, it wouldn't be necessary. Definitely don't double up the same content in both attributes. And if what you'd put in the image's alt attribute is already in text alongside the image, then just leave the alt blank as otherwise screenreaders would have to listen to the same piece of text read out twice.
- You've spelt restaurant wrong in the title tag. Also in the title tags, you've abbreviated J.B. but it's spelt in full elsewhere; it should be consistent.
- You've got no h1 on the home page. On the home page, I'd wrap the logo (whose alt attribute would be the name of the site) in a h1. On internal pages, replace it with a div as you have currently.
- You've currently got h3s in the main body and a h2 in the footer which means you've skipped a heading level (two if you don't have a h1). Make the h3s in the body h2s and just style them to the required size. I wouldn't use a h2 for the address details in the footer - use Microformats instead.
- On internal pages, leftside and rightside are more presentational than they should be as ID names.
- On the dining page, the names of the courses and their description should be in separate elements. The headings should be in heading tags.
- The images in the left col should be bg-images. If they were in the HTML, leaves the alts blank as they're decoration, not content. Alts on the ones in the right column is OK as they are content.
- The left column ending before the rest of the content on the history page looks a bit odd.
Wed, 2010-02-17 04:10
#7
WOW! This is great, exactly
WOW! This is great, exactly what I was looking for.
Thanks a lot Tyssen for your thorough review and feedback, this is very much appreciated.
I forgot the h1 on the logo... doh
I'll make these updates tomorrow.
Tue, 2010-03-16 10:47
#8
I think you could enlarge
I think you could enlarge your images a little bit so that the text on them could be read