Go Back   CodingForums.com > :: Client side development > General web building > Site reviews

Before you post, read our: Rules & Posting Guidelines

Reply
 
Thread Tools Rate Thread
Enjoy an ad free experience by logging in. Not a member yet? Register.
Old 05-29-2008, 09:00 PM   PM User | #1
jcdevelopment
Senior Coder

 
jcdevelopment's Avatar
 
Join Date: Oct 2007
Location: Cowboy Nation
Posts: 2,171
Thanks: 173
Thanked 257 Times in 257 Posts
jcdevelopment will become famous soon enoughjcdevelopment will become famous soon enough
review: theweddingpartyevent.com

Just finished this new site up, for a whopping $200 dollars (a friend). Just wanted some feedback on the design and code. Thanks all!

http://theweddingpartyevent.com/index.html
jcdevelopment is offline   Reply With Quote
Old 05-29-2008, 09:29 PM   PM User | #2
BoldUlysses
Regular Coder

 
BoldUlysses's Avatar
 
Join Date: Jan 2008
Location: Winston-Salem, NC
Posts: 938
Thanks: 10
Thanked 190 Times in 187 Posts
BoldUlysses is on a distinguished road
Seeing as how most of the pages don't contain a great deal of content, it would seem the horizontal size of the site is driven more by the constraints of the top menu. My displays are 1280 x 1024 and it barely fits on one. Thought about putting some of the menu items (contact us, press, etc) on the bottom bar to narrow things up?

It might be helpful to add a

Code:
cursor:pointer;
to some of the links.

Those are my only comments on design... I'm sure someone will chime in about the code.

Looks great!
BoldUlysses is offline   Reply With Quote
Old 05-29-2008, 09:32 PM   PM User | #3
binaryWeapon
Regular Coder

 
Join Date: Sep 2007
Location: AZ, USA
Posts: 685
Thanks: 6
Thanked 46 Times in 46 Posts
binaryWeapon is on a distinguished road
Technical:
- If your screen isn't big enough, the menu wraps to the next line, but the background of the menu doesn't, so it's hard to read
- IMO the logo should say "The Wedding Part" or something like that

Design
- On all pages except the index page, the text is right up flush against the picture of the bride, and the bullets appear on top of that image

Nice site.
binaryWeapon is offline   Reply With Quote
Old 05-29-2008, 09:38 PM   PM User | #4
jcdevelopment
Senior Coder

 
jcdevelopment's Avatar
 
Join Date: Oct 2007
Location: Cowboy Nation
Posts: 2,171
Thanks: 173
Thanked 257 Times in 257 Posts
jcdevelopment will become famous soon enoughjcdevelopment will become famous soon enough
Well, thats my ignorance palying a roll, i didnt even think about the resolution problem. I will go back and see if i can adjust a few things around. Thanks for the feedback so far guys.

EDIT**

Now that i look at it in 1024x768 the press button shifts to the lower area, i will move that button!

Last edited by jcdevelopment; 05-29-2008 at 09:40 PM.. Reason: realization
jcdevelopment is offline   Reply With Quote
Old 05-29-2008, 10:47 PM   PM User | #5
harbingerOTV
Senior Coder

 
Join Date: Jan 2005
Location: Memphis, TN
Posts: 1,769
Thanks: 8
Thanked 127 Times in 125 Posts
harbingerOTV will become famous soon enough
Looks like you need a min-width of 1084px. kinda wide but oh well. I set your content area to about 90%+/- so that the right side of the text has some room. certain words cram up against it.

Maybe drop your right side headings down a few more pixels. Also the pic at top (can't remember what they are called), maybe repeat it inverted on the bottom as well?
__________________
Stop making things so hard on yourself.
i is tugbucket :: help raise tugburg :: Whitehaven Kiwanis
harbingerOTV is offline   Reply With Quote
Old 05-29-2008, 10:51 PM   PM User | #6
tomws
Senior Coder

 
tomws's Avatar
 
Join Date: Nov 2007
Location: Arkansas
Posts: 2,644
Thanks: 29
Thanked 330 Times in 326 Posts
tomws will become famous soon enoughtomws will become famous soon enough
I like it.

Is the page title supposed to be so long? Looks like keywords. The Sponsors page title is different from the rest.

If the other guys wouldn't have mentioned the nav folding under itself, I wouldn't have noticed on FF because the the missing light-colored background. By the way, when it does fold under, Firefox crams the left image a bit under the main text such that left-padding on the text vanishes. On IE7, the rightmost parts of the nav just disappear on smaller windows. IE6 folds under like FF, but keeps a light background.

Past the first page, I wonder why no top logo. Almost feels like something missing.

But again, I like. Clean and simple. Easy on my eyes.
tomws is offline   Reply With Quote
Old 05-29-2008, 11:01 PM   PM User | #7
Apostropartheid
The Apostate


 
Apostropartheid's Avatar
 
Join Date: Oct 2007
Posts: 3,215
Thanks: 16
Thanked 265 Times in 263 Posts
Apostropartheid is on a distinguished road
I like the colour scheme. Binary's right, you do need some identifying marker on the logo (at least making it stand out from the background more), as it just looks like a background at the moment. Also, what on god's earth has the favicon got to do with the rest of the site?

It's interesting that you chose to use inline styles. Don't. Also look into shorthand CSS properties like font and background: by the bloatedness of the event.css, it looks like you'll save a lot of time with these. I spent half an hour trying to figure out where the body text was getting its font declarations from and I discovered that you...don't have any?

You have rather single-mindedly designed for a rather high screen resolution. Do you really need that many links at the top? Really? If so, maybe you could look into a dropdown solution. Remember that the majority of people don't browse full-screen: you have a lot less real-estate than you care to think.

Anyway, it's nice to see some tasteful colours applied. Most of the fonts are lovely, but you've overused italics a bit sometimes. It's generally strong, and it's nice (I feel weak saying that, but it's true.) Keep it up.
__________________
Blog | Twitter
Useful links: W3C HTML Validator | W3C CSS Validator | HTML 5 Guide
CF: HTML & CSS Resources/Tutorials Thread | HTML & CSS Posting Rules and Guidelines
Remember: no link, no code, no help!
Apostropartheid is offline   Reply With Quote
Old 05-29-2008, 11:21 PM   PM User | #8
jcdevelopment
Senior Coder

 
jcdevelopment's Avatar
 
Join Date: Oct 2007
Location: Cowboy Nation
Posts: 2,171
Thanks: 173
Thanked 257 Times in 257 Posts
jcdevelopment will become famous soon enoughjcdevelopment will become famous soon enough
Thanks again guys, here is the thing..

Quote:
Do you really need that many links at the top? Really?
haha, yeah.. that would be the "bosses" wants and needs there.

Quote:
I spent half an hour trying to figure out where the body text was getting its font declarations from and I discovered that you...don't have any?
..ooops, i forget to assign them, i just forgot. I will get on that thank you.

Well i have a few extra things to look at now, this is really my first or second design so i am learning. Thanks every one!
jcdevelopment is offline   Reply With Quote
Old 05-30-2008, 10:50 AM   PM User | #9
fside
Regular Coder

 
Join Date: Mar 2008
Posts: 301
Thanks: 2
Thanked 30 Times in 30 Posts
fside is an unknown quantity at this point
I would second 'cyan'. Use just a standard left links menu. And I would also try for a little darker background for more contrast, and more specification about font styles, spacing, etc.
fside is offline   Reply With Quote
Reply

Bookmarks

Jump To Top of Thread


Thread Tools
Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT +1. The time now is 09:03 PM.


Advertisement
Log in to turn off these ads.