8 replies [Last post]
purewebdesigner
purewebdesigner's picture
Offline
Enthusiast
Washington DC
Last seen: 2 years 35 weeks ago
Washington DC
Timezone: GMT-5
Joined: 2007-04-22
Posts: 234
Points: 95

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

CSSExpression EngineHTMLWordpressCoffee

Personal Website: curtisscott.com
Follow Me on Twitter: twitter.com/curtisscott_com

purewebdesigner
purewebdesigner's picture
Offline
Enthusiast
Washington DC
Last seen: 2 years 35 weeks ago
Washington DC
Timezone: GMT-5
Joined: 2007-04-22
Posts: 234
Points: 95

purewebdesigner wrote: Please

purewebdesigner wrote:

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/

CSSExpression EngineHTMLWordpressCoffee

Personal Website: curtisscott.com
Follow Me on Twitter: twitter.com/curtisscott_com

Hugo
Hugo's picture
Offline
Moderator
London
Last seen: 6 years 29 weeks ago
London
Joined: 2004-06-06
Posts: 15668
Points: 2806

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)

Before you make your first post it is vital that you READ THE POSTING GUIDELINES!
----------------------------------------------------------------
Please post ALL your code - both CSS & HTML - in [code] tags
Please validate and ensure you have included a full Doctype before posting.
Why validate? Read Me

Verschwindende
Verschwindende's picture
Offline
Guru
Last seen: 13 weeks 4 days ago
Timezone: GMT-4
Joined: 2009-10-09
Posts: 2050
Points: 2282

Hugo wrote: ... Haven't

Hugo wrote:

... 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. Tongue

PWD, I'm very jealous of your design skills. Looks great. Are you local to that place?

purewebdesigner
purewebdesigner's picture
Offline
Enthusiast
Washington DC
Last seen: 2 years 35 weeks ago
Washington DC
Timezone: GMT-5
Joined: 2007-04-22
Posts: 234
Points: 95

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. Smile 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.

CSSExpression EngineHTMLWordpressCoffee

Personal Website: curtisscott.com
Follow Me on Twitter: twitter.com/curtisscott_com

scept1c
scept1c's picture
Offline
Regular
Last seen: 9 years 10 weeks ago
Joined: 2010-02-01
Posts: 16
Points: 17

I would replace the "brown"

I would replace the "brown" text to another color.

http://www.cms-informer.com/ PHP Content management Systems Blog and News
Articles, Information, Documentation, Blogs

Tyssen
Tyssen's picture
Offline
Moderator
Brisbane
Last seen: 6 years 41 weeks ago
Brisbane
Timezone: GMT+10
Joined: 2004-05-01
Posts: 8201
Points: 1386

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.

How to get help
Post a link. If you can't post a link, jsFiddle it.
My blog | My older articles | CSS Reference

purewebdesigner
purewebdesigner's picture
Offline
Enthusiast
Washington DC
Last seen: 2 years 35 weeks ago
Washington DC
Timezone: GMT-5
Joined: 2007-04-22
Posts: 234
Points: 95

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.

CSSExpression EngineHTMLWordpressCoffee

Personal Website: curtisscott.com
Follow Me on Twitter: twitter.com/curtisscott_com

scept1c
scept1c's picture
Offline
Regular
Last seen: 9 years 10 weeks ago
Joined: 2010-02-01
Posts: 16
Points: 17

I think you could enlarge

I think you could enlarge your images a little bit so that the text on them could be read

http://www.cms-informer.com/ PHP Content management Systems Blog and News
Articles, Information, Documentation, Blogs