6 replies [Last post]
bpat1434
bpat1434's picture
Offline
Enthusiast
Last seen: 13 years 46 weeks ago
Timezone: GMT-5
Joined: 2004-09-17
Posts: 127
Points: 0

Hey guys. I'm just looking for your thoughts on the design and layout. The most CSS intensive stuff is the photographs area, but the site is 100% validated (althought I don't show the graphics) xhtml and css.

http://www.ridgeswimclub.org

~Brett

~Brett

Anonymous
Anonymous's picture
Guru

Re: Your Thoughts

bpat1434 wrote:
The most CSS intensive stuff is the photographs area, but the site is 100% validated (althought I don't show the graphics) xhtml and css.
Not from what I see. Not even the splash page validates.

bpat1434
bpat1434's picture
Offline
Enthusiast
Last seen: 13 years 46 weeks ago
Timezone: GMT-5
Joined: 2004-09-17
Posts: 127
Points: 0

Your Thoughts

Well, it does validate CSS & XHTML if I strictly use the tags but not inside of the php page. It gives me errors, but the code is good. So perhaps it's not 100%, but it's as close to valid as it's gonna get for right now.

Any other comments other than its validity?

~Brett

~Brett

antibland
antibland's picture
Offline
Leader
Pittsburgh
Last seen: 12 years 46 weeks ago
Pittsburgh
Joined: 2005-01-17
Posts: 603
Points: 0

sink or swim

I think the site would benefit well from a drop shadow around the left and right content edges, to give it some depth. You already have a #wrapper div surrounded by (ulp) <center></center>. Just apply the bg image on that element with the necessary shadowing (or not).

background: #fff url(somebg.gif) 0 0 repeat-direction
;

Other than that, the page looks all right. You seriously should validate this code when you have the time. It matters more than you think.

Antibland

bpat1434
bpat1434's picture
Offline
Enthusiast
Last seen: 13 years 46 weeks ago
Timezone: GMT-5
Joined: 2004-09-17
Posts: 127
Points: 0

Your Thoughts

Cool, thanks for the thoughts.

Why the "(ulp)" (which I'm guessing is *gulp) with my center tags? Setting the body{ text-align: center;} didn't work for me so I had to resort to using them.

~Brett

~Brett

antibland
antibland's picture
Offline
Leader
Pittsburgh
Last seen: 12 years 46 weeks ago
Pittsburgh
Joined: 2005-01-17
Posts: 603
Points: 0

Your Thoughts

The center tag is a deprecated html tag, and should not be used. To center a site on any modern browser, you can do the following. You already have a #wrapper div, so this should be easy to fix.

body { text-align: center; } 
#wrapper { margin: 0 auto; }

Antibland

EOBeav
EOBeav's picture
Offline
Regular
Pendleton, Oregon
Last seen: 15 years 9 weeks ago
Pendleton, Oregon
Timezone: GMT-7
Joined: 2004-08-06
Posts: 49
Points: 0

Re: Your Thoughts

bpat1434 wrote:
Hey guys. I'm just looking for your thoughts on the design and layout. The most CSS intensive stuff is the photographs area, but the site is 100% validated (althought I don't show the graphics) xhtml and css.

http://www.ridgeswimclub.org

~Brett

Lose the intro page. You're just creating another layer of space between your visitors and the information you want them to see. One click might not seem like much, but you only have a few seconds to get your intiial message across before people start clicking away from your site. Don't waste those seconds on an intro page. From what I can tell, however, most of your visitors will be community people specifically looking for some information before they click away: When is the pool open? How much does it cost? Can we bring something to eat? Stuff like that.

Other than that, your layout is good, and I like your menu. Get it to validate and I think you've got a nice-looking, functional website.

Get Thunderbird!