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.
Page 1 of 3 123 LastLast
Results 1 to 15 of 34
  1. #1
    New Coder
    Join Date
    Dec 2008
    Posts
    59
    Thanks
    3
    Thanked 0 Times in 0 Posts

    Review http://tinyurl.com/4875x7

    I am customizing this wordpress theme for my cousin's website but seems to be missing something.

    Your thoughts??

    Thanks and happy holiday

  • #2
    Senior Coder Rowsdower!'s Avatar
    Join Date
    Oct 2008
    Location
    Some say it's everything.
    Posts
    2,027
    Thanks
    5
    Thanked 397 Times in 390 Posts
    Hi Reta, welcome to the forums.

    I'll go ahead and post a link to the site in question for anyone who is as lazy as I am...

    A good place to start in any case will be to fix the validation errors in your code.

    As for the aesthetics, the logo seems to flow well enough with the design with the possible exception of the main content div. That stark white background may be too much contrast to handle. The right div looks bare after the ad and search box. Maybe you could just let the content flow underneath those sections (essentially plop that div inside the main content div and switch to a 2-column layout). There's no need for a third column if you don't have anything to put in it.

    Those are the only main issues that I can think of really.
    The object of opening the mind, as of opening the mouth, is to shut it again on something solid. –G.K. Chesterton
    See Mediocrity in its Infancy
    It's usually a good idea to start out with this at the VERY TOP of your CSS: * {border:0;margin:0;padding:0;}
    Seek and you shall find... basically:
    validate your markup | view your page cross-browser/cross-platform | free web tutorials | free hosting

  • #3
    New Coder
    Join Date
    Dec 2008
    Posts
    59
    Thanks
    3
    Thanked 0 Times in 0 Posts
    Hey Rowsdower!,

    Thanks you for your thoughts and suggestions. I will look into the validation errors now and try and fix them.

    We actually plan on using the third column for recent comments, top selling product, ads, etc.

    I made a few changes based on your suggestions.

    Let me know what you think of it now.

    Thanks,

    Reta

  • #4
    New Coder
    Join Date
    Dec 2008
    Posts
    59
    Thanks
    3
    Thanked 0 Times in 0 Posts
    Got rid of all the warnings and some errors. The numbers are pretty low compared to well known websites.

  • #5
    Senior Coder gnomeontherun's Avatar
    Join Date
    Sep 2007
    Location
    Houston
    Posts
    2,846
    Thanks
    10
    Thanked 238 Times in 229 Posts
    First thing is the contrast. My eyes don't flow anywhere, they just start to hurt. Soften the black to a dark gray and maybe take the white up a smidgeon. The blue would be fine if you do that, I think, but it might need a little adjustment.

    I don't like the HR bars, they don't fit with the sidelines. Perhaps you can work with those so they don't look so blocky?

    Typography, almost everything is set in the same font. Maybe some research on this will help you see some other options so its not all the same.

    Reflections are cool, but it seems too much in this case. Most of the time the reflection is not a complete copy of the main image, rather it fades out completely 2/3rds the way down. I think this would help.

    Your navigation doesn't scream "I'm the navigation" I think because the background just flows with everything else. Perhaps you can think of a way to set it apart a little bit.

    Fade the R in the main post background more.
    jeremy - gnomeontherun
    Educated questions often get educated answers, and simple questions often get simple answers.

  • #6
    New Coder
    Join Date
    Dec 2008
    Posts
    59
    Thanks
    3
    Thanked 0 Times in 0 Posts
    jeremywilken,

    Appreciate the time

    Made some minor changes base on what you have said so far. I will look into further customizing the theme later on. I'm finding it really difficult change the navigation. Not very good with the coding.

    Thanks

    Reta
    Last edited by Reta; 12-20-2008 at 09:19 PM.

  • #7
    Senior Coder twodayslate's Avatar
    Join Date
    Mar 2007
    Location
    VA
    Posts
    1,042
    Thanks
    67
    Thanked 39 Times in 39 Posts
    ads are bad.
    reflection takes up too much space.
    validate
    twitter | Quality Hosting - $5.95/mo*
    Feel free to PM me!

  • #8
    Senior Coder Rowsdower!'s Avatar
    Join Date
    Oct 2008
    Location
    Some say it's everything.
    Posts
    2,027
    Thanks
    5
    Thanked 397 Times in 390 Posts
    I still say the white is too harsh for the main content. Also, when the main content is very short (for example in the shopping cart page when no items are in your cart) your page looks sort of awkward. You may want to play around with a minimum height to the div - maybe even full-height all the time - to keep it from looking so naked (it might not help at all, but see if you like it).

    Also, I didn't add anything to my cart the first time through, but now I am noticing a severe lack of contrast between the left banner cart summary and its background. The individual items show up OK, but the totals and item count are very dark gray and the background is black. You may want to give the cart summary its own light-ish background to make the text more visible. Either that or lighten up the font color significantly.

    As for the validation errors (currently 19) four of these are misuses of an ID (using the same ID, such as postinfo, more than once). If you truly need the ID more than once you should just change these to a class instead. Besides, if you intend to do any java scripting on these page elements you'll need to sort these out sooner or later anyway. It'll be easier to do it now while it's all fresh... Another four or so are uses of "FONT" in your markup which could easily be fixed with CSS. The rest appear to be tag errors which could be serious issues with your layout. It's great that you have reduced your errors, but you really shouldn't ignore the ones that remain.
    The object of opening the mind, as of opening the mouth, is to shut it again on something solid. –G.K. Chesterton
    See Mediocrity in its Infancy
    It's usually a good idea to start out with this at the VERY TOP of your CSS: * {border:0;margin:0;padding:0;}
    Seek and you shall find... basically:
    validate your markup | view your page cross-browser/cross-platform | free web tutorials | free hosting

  • #9
    New Coder
    Join Date
    Dec 2008
    Posts
    59
    Thanks
    3
    Thanked 0 Times in 0 Posts
    I changed the navi and background a fixed a few things here and there.

    Your thoughts plz

    thanks

    You guys have been a great help so far.
    Last edited by Reta; 12-26-2008 at 12:50 AM.

  • #10
    Senior Coder twodayslate's Avatar
    Join Date
    Mar 2007
    Location
    VA
    Posts
    1,042
    Thanks
    67
    Thanked 39 Times in 39 Posts
    no ads. put nav on one line. ten billion other things...
    twitter | Quality Hosting - $5.95/mo*
    Feel free to PM me!

  • #11
    New Coder
    Join Date
    Dec 2008
    Posts
    59
    Thanks
    3
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by twodayslate View Post
    no ads. put nav on one line. ten billion other things...
    Twodayslate,

    Thanks for taking the time.

    We will be selling space for ads. My client likes the idea of the nav being on top. Just looking to improve the overall idea.

  • #12
    Senior Coder twodayslate's Avatar
    Join Date
    Mar 2007
    Location
    VA
    Posts
    1,042
    Thanks
    67
    Thanked 39 Times in 39 Posts
    Quote Originally Posted by Reta View Post
    Twodayslate,

    Thanks for taking the time.

    We will be selling space for ads. My client likes the idea of the nav being on top. Just looking to improve the overall idea.
    the nav on top is good. just put everything in the nav on one line.

    is this your first site? and you are getting paid for it? take it slowly man and make it good. GL
    twitter | Quality Hosting - $5.95/mo*
    Feel free to PM me!

  • #13
    New Coder
    Join Date
    Dec 2008
    Posts
    59
    Thanks
    3
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by twodayslate View Post
    the nav on top is good. just put everything in the nav on one line.

    is this your first site? and you are getting paid for it? take it slowly man and make it good. GL
    Not sure what you mean by one line. I'm no expert. I'm not getting paid as of yet. Depends on the success of the site :-) Thanks

  • #14
    Senior Coder twodayslate's Avatar
    Join Date
    Mar 2007
    Location
    VA
    Posts
    1,042
    Thanks
    67
    Thanked 39 Times in 39 Posts
    perhaps we are seeing different things...
    http://img247.imageshack.us/img247/5...enshot2ik9.png
    twitter | Quality Hosting - $5.95/mo*
    Feel free to PM me!

  • #15
    New Coder
    Join Date
    Dec 2008
    Posts
    59
    Thanks
    3
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by twodayslate View Post
    perhaps we are seeing different things...
    http://img247.imageshack.us/img247/5...enshot2ik9.png
    Whoa! That is disgusting. What browser are you using? I tried looking at the site from 2 diff browsers on 2 different screens with different resolutions and they appear this way:



    [img=http://img224.imageshack.us/img224/599/ssos7.th.jpg]

    http://img224.imageshack.us/img224/599/ssos7.jpg


  •  
    Page 1 of 3 123 LastLast

    Posting Permissions

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