Review Request: carlwinslow dot com

They have: 5,633 posts

Joined: Jan 1970

I got the layout I want up and running. I think I got all the errors out of it. Please give me your honest opinion. Ideas, suggestions, critiques welcome. Site is about 2 months old, all content purely ours. Thanks!

http://www.carlwinslow.com

The Webmistress's picture

She has: 5,586 posts

Joined: Feb 2001

Welcome to TWF Harley,

While I'm looking at the site can you please review some of the other sites posted here in the critique section as per the posting agreement which will not only help others but give you ideas as well.

Julia - if life was meant to be easy Michael Angelo would have painted the floor....

They have: 344 posts

Joined: Jun 2002

Not a bad site there!

It's a bit too grey for me but it still looks nice!

Where you have all the banners I suggest that you separate them a bit and maybe add some light effects like on, http://cmdirector.neosurge.net ? It would definately add a nice touch to the site.

I would adjust the size of all your right hand size tables too so that it fits onto one page and how about changing it so the table header is centered?

Also, the text in the middle is a bit too large for me but that is my opinion. Good luck with your site mate. It looks good!

disaster-master's picture

She has: 2,154 posts

Joined: May 2001

Fix the structure of you document. You have the doctype below the tag. The Doctype should be listed first. Below is a basic example of how things should be arranged.
See structural templates for more info.

Title

In your style sheet, you can add margin : 0px; to the body and that should take care of the horizontal scroll.

Add some cellpadding to the tables so that the text won't be jammed up against the border of each section. You can also add padding in your style sheet but that may make your horizontal scroll come back.

You should use your style sheet to control the font sizes intead of using the font tags.

I think I would also add underline on hover on your links so that they will show up better.
a:hover{color : #990000; text-decoration : underline;}

Good job. You are getting there. Smiling

Sonia

They have: 26 posts

Joined: Mar 2003

Excellent site! I love the layout and color scheme! One thing I would fix as mentioned by someone else is resize your tables, so you eliminate the need to scroll to the right. Other than that it’s easy to navigate and looks real clean, nice work.

Mike Caskanette.
LifeLine Aquarium Systems.
http://www.lifelineaquarium.com

Megan's picture

She has: 11,421 posts

Joined: Jun 1999

I think this looks nice but the colour scheme and layout don't seem to be particularly appropriate for the site content here. It seems like you've got a fun, lighthearted theme going here, so why is it all drab grey? Put some colour in there! I often say that when you can get a way with going funky, DO IT, because there aren't many opportunities in this business to really get creative.

I also think that the boxy layout isn't doing a whole lot for you. You don't need to link every page up on the front page. Instead, create intermediary pages for each section, which would provide a setting and link to individual articles. That would make it easier to navigate the site since you'd have one consolidated navbar which would make it easy for people to quickly scan the links and get an idea of what they can do on your site. Oh, and put some padding in those boxes - the links shouldn't but up against the edges like that.

They have: 62 posts

Joined: Jan 2002

Well first I have to tell you, you have the funniest page header I've seen in awhile. I don't know what it means... but it did make me laugh. Laughing out loud

I do agree that you shouldn't set your tables on a fixed width. Try setting your outside table to a variable size. (I like 98% myself, but you could go with any percentage value you want) You can set your left and right side tds to a fixed position (?px) and have the center td set to 100%. This would give you a more fluid layout and take care of that nasty scrollbar problem.

I also agree that you shouldn't have all your links on your front page. Seperate your content into main sections (which you've done) and just link to those section pages. Then you can have sub navigation for each section.
(btw: I have a site where I do have all my links on my first page using drop down navigation. The problem is the drop downs are not very attractive)

Coding problems:
Your Doctype needs to go before the html

You have a meta tag in the middle of your page. That needs to be moved up to the head section.

You have some links pointing to local files on your computer. You want to check to make sure they're all pointing to their location on your site.

I'm not sure what this is: xthumbnail-orig-image="images/cow on 2 legs.jpg"' maybe someone else knows.

Small problem... you have width and height attributes in a lot of your a href elements. That's not the place for them.

You've got a couple closing tags missing, which can be a problem with some browsers and there is no closing tags for body or html.
Did those get accidently deleted?

You've got some neat stuff here.
Keep having fun with it all! Laughing out loud

The purpose of education is to
get more jokes.

Want to join the discussion? Create an account or log in if you already have one. Joining is fast, free and painless! We’ll even whisk you back here when you’ve finished.