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 TheShaner's Avatar
    Join Date
    Sep 2005
    Location
    Orlando, FL
    Posts
    1,126
    Thanks
    2
    Thanked 40 Times in 40 Posts

    Site Review Please

    This is a site I made for my friend's mom who sells Italian Charms. Please don't comment on how the business is run. She wants to only process orders over the phone, which is very inconvenient I know, but she mainly wanted the site for her personal customers and referrals. Anyway, the site is:

    http://peggyscharms.com

    I welcome any feedback. I know it's fairly amateurish, but it's my first attempt using all CSS instead of tables. Plus, I'm not artistic in the least bit, so coming up with any kind of a design is a miracle for me, lol. Graphics have always been the hardest for me, so any suggestions on how to improve them would be appreciated.

    Also, comments on the coloring scheme would be good. Sure, I can match the clothes on my body somewhat, but web design color schemes is a bit more difficult for me, hehe.

    One more note: I realize that using all pixel sizes for everything, mainly font size, is not looked upon well, so I'm hoping to change that eventually. I know I should be using ems and percentages. I just need to figure out how to make it work with my graphics.

    Thanks for any critiques!

    -Shane

  • #2
    Regular Coder hinch's Avatar
    Join Date
    Sep 2005
    Location
    UK
    Posts
    923
    Thanks
    25
    Thanked 80 Times in 80 Posts
    not bad i worship you for the ability to get css to even slightly behave

    colour code is quite bad though light colours all mixed together not enough contrast i'm colour blind so got kinda lost on a few pages.

    the listings page when you for example go here http://peggyscharms.com/charms.php?category=letters is quite hard to read dodgy fonts/too small + white on light blue

    your shopping cart page needs some serious tidying up and standardisation of fonts to match the rest of the site.

    all in all though theyre just superficial changes the functionality is there its nice and fast and in general everything is where it should be.

    I have to say though being italian born and with the majority of my family living over there I haven't heard of "italian charms" before oO
    A programmer is just a tool which converts caffeine into code

    My work: http://www.fcsoftware.co.uk && http://www.firstcontactcrm.com
    My hobby: http://www.angel-computers.co.uk
    My life: http://www.furious-angels.com

  • #3
    Senior Coder TheShaner's Avatar
    Join Date
    Sep 2005
    Location
    Orlando, FL
    Posts
    1,126
    Thanks
    2
    Thanked 40 Times in 40 Posts
    Thanks for your comments Hinch. Appreciate it as always! I knew the color coding was bad. Having trouble finding the right contrast. I'll keep working at it though.

    I definitely have to get fonts to match all around, esp. the shopping cart, like you said.

    As for never hearing of Italian charms, I can't tell you where they started, hehe, but they're quite popular over here. Google it and you'll find many sites devoted to them. However, it may be one of those things where it didn't actually get started in Italy, but somehow they credit for it. Sort of like pizza lol

    -Shane

  • #4
    Senior Coder TheShaner's Avatar
    Join Date
    Sep 2005
    Location
    Orlando, FL
    Posts
    1,126
    Thanks
    2
    Thanked 40 Times in 40 Posts
    Well, i changed the side bar font to match the navbar font. All the other text match completely, including all cart tables and the footer.

    I changed the links in the content area to stand out more, along with the navbar links to a better color and the footer links darker. I think the white looks fine in the side bar, but maybe I should darken it up to a light gray for more contrast.

    The Charms title for the side bar will be changed to the same color of the navbar once I get home and can edit my PSD. The yellow seems to strain the eyes, whereas the pinkish purple on the navbar seems better suited and contrast better with the background.

    Don't know if there's anything else I should change, esp. graphics-wise. It still seems amateurish in look. I'm guessing that is due to the basic design and graphics. Wish I knew what I could do to give it a more professional look.

    Thanks Hinch and any future comments from anyone!

    -Shane

  • #5
    Regular Coder ubik's Avatar
    Join Date
    Oct 2005
    Location
    westCoast, usa
    Posts
    347
    Thanks
    0
    Thanked 0 Times in 0 Posts
    nice site, when i try to add something to the shopping cart i getthis back, in both internet explorer & firefox:

    Warning: mysql_query(): Access denied for user: 'peggyscharms@localhost' (Using password: NO) in /var/www/html/minibasket.inc on line 67

    Warning: mysql_query(): A link to the server could not be established in /var/www/html/minibasket.inc on line 67

    Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /var/www/html/minibasket.inc on line 68
    "True knowledge exists in knowing that you know nothing."
    -Socrates

  • #6
    Regular Coder
    Join Date
    Apr 2005
    Posts
    388
    Thanks
    0
    Thanked 0 Times in 0 Posts
    good job, I like it. I have to agree with hinch about the colors but hey whats in a color.

  • #7
    Senior Coder TheShaner's Avatar
    Join Date
    Sep 2005
    Location
    Orlando, FL
    Posts
    1,126
    Thanks
    2
    Thanked 40 Times in 40 Posts
    I've tweaked the colors a bit, but I still have to do a little more (like I've yet to change the Charms image color on the sidebar).

    As for the shopping cart, you must've caught me on the day I was changing around a bunch of stuff, hehe. It works fine now.

    One difference I've put in there is that it now only shows 12 charms at a time and then places a Next and Previous link to go through the rest. Should I stick with this or do you think people prefer just to be able scroll down and see everything on 1 page?

    For the code I created for the above effect, I end up doing a search EACH time a Next or Previous is clicked. I was hoping to come up with a way to only do the search the first time. The Next and Previous pages could just use a variable holding all the rows and passed back and forth using sessions. But I'm a little stuck on what I could do to determine whether to keep using the session or to clear it when a new category of charms is selected. I believe I could just store the category in a session also and if the category changes, then clear the session holding the rows? I'll have to tinker with this. Any other suggestions would be appreciated.

    Thanks for the reviews!

    -Shane

  • #8
    Regular Coder hinch's Avatar
    Join Date
    Sep 2005
    Location
    UK
    Posts
    923
    Thanks
    25
    Thanked 80 Times in 80 Posts
    much better now although in general the design just isnt very professional but then i'm used to looking at big it firm websites not little trinket sites so ignoring me is probably best
    A programmer is just a tool which converts caffeine into code

    My work: http://www.fcsoftware.co.uk && http://www.firstcontactcrm.com
    My hobby: http://www.angel-computers.co.uk
    My life: http://www.furious-angels.com

  • #9
    Senior Coder TheShaner's Avatar
    Join Date
    Sep 2005
    Location
    Orlando, FL
    Posts
    1,126
    Thanks
    2
    Thanked 40 Times in 40 Posts
    Yeah, if this was for a big business, I'd trash it instantly, but it's just for someone's small side business with hardly any traffic. Although, I'd still like to make it look a bit more professional than it is now. I'm just not sure how to. I'm a programmer, not a designer, haha. Thanks for the comments! I really appreciate it.

    -Shane


  •  

    Posting Permissions

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