Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Results 1 to 9 of 9
  1. #1
    Senior Coder jcdevelopment's Avatar
    Join Date
    Oct 2007
    Location
    Cowboy Nation
    Posts
    2,179
    Thanks
    174
    Thanked 257 Times in 257 Posts

    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

  • #2
    Regular Coder BoldUlysses's Avatar
    Join Date
    Jan 2008
    Location
    Winston-Salem, NC
    Posts
    938
    Thanks
    10
    Thanked 190 Times in 187 Posts
    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!

  • #3
    Regular Coder
    Join Date
    Sep 2007
    Location
    AZ, USA
    Posts
    685
    Thanks
    6
    Thanked 46 Times in 46 Posts
    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.

  • #4
    Senior Coder jcdevelopment's Avatar
    Join Date
    Oct 2007
    Location
    Cowboy Nation
    Posts
    2,179
    Thanks
    174
    Thanked 257 Times in 257 Posts
    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

  • #5
    Senior Coder
    Join Date
    Jan 2005
    Location
    Memphis, TN
    Posts
    1,785
    Thanks
    8
    Thanked 131 Times in 129 Posts
    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

  • #6
    Senior Coder tomws's Avatar
    Join Date
    Nov 2007
    Location
    Arkansas
    Posts
    2,644
    Thanks
    29
    Thanked 330 Times in 326 Posts
    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.

  • #7
    The Apostate Apostropartheid's Avatar
    Join Date
    Oct 2007
    Posts
    3,215
    Thanks
    16
    Thanked 265 Times in 263 Posts
    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.

  • #8
    Senior Coder jcdevelopment's Avatar
    Join Date
    Oct 2007
    Location
    Cowboy Nation
    Posts
    2,179
    Thanks
    174
    Thanked 257 Times in 257 Posts
    Thanks again guys, here is the thing..

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

    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!

  • #9
    Regular Coder
    Join Date
    Mar 2008
    Posts
    301
    Thanks
    2
    Thanked 30 Times in 30 Posts
    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.


  •  

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •