di11on Posted September 4, 2006 Share Posted September 4, 2006 Hey folks, Put this Vintage Clothing website together for my sister... would be grateful for some feedback. www.vintagevibrations.com Go easy on me :-) Thanks folks, Jody Link to comment Share on other sites More sharing options...
bows2babydolls Posted September 4, 2006 Share Posted September 4, 2006 ooh very snazzy! it still looks out of the box but thats not such a problem, its a nice and fresh design. I love the wee flowers next to each category :) I just love this site! can't find anything wrong with it. Sorry for not being helpful Link to comment Share on other sites More sharing options...
Jack_mcs Posted September 4, 2006 Share Posted September 4, 2006 Just a few things to start off: - remove the top space abvoe the header. That is wasted space, which you can't afford to lose. - I would make the image move visible. The visitor has to scroll down to see the first one (see above). - convert the top->catalog in the breadcrumb to Home. You aren't using themboth and they are hurting your Page Rank by having them both there. - your ebay link is broken. Jack Support Links: For Hire: Contact me for anything you need help with for your shop: upgrading, hosting, repairs, code written, etc. All of My Addons Get the latest versions of my addons Recommended SEO Addons Link to comment Share on other sites More sharing options...
tastar Posted September 4, 2006 Share Posted September 4, 2006 Very nice site! Very well done with a great mix of contributions. You do have at least one mechanical problem (in addition to the eBay problem previously mentioned). If you add the star product to the shopping cart and hit Continue, my browser got lost and errored out. Also, when I added the star product to a cart with other products, the other products disappeared. And, it seems that when you use the Buy Now button from the product listings, it empties the cart. The Add-to-Cart button in individual product listings does seem to work. In any case, you have shopping cart issues. Maybe you could put your star product in a top of the column right column info box (below the cart box). That would pull your featured products up so that they would be more visible. And, the star product would then be visible on all of your pages rather than just the home page. Still, a wonderful job with a few issues to correct. Tony Link to comment Share on other sites More sharing options...
jameshontishar Posted September 4, 2006 Share Posted September 4, 2006 Hey folks, Put this Vintage Clothing website together for my sister... would be grateful for some feedback. www.vintagevibrations.com Go easy on me :-) Thanks folks, Jody Jody I think you have done really well are you a beginner, as i am an oscommerce virgin and just putting my site together, and would love to know how you went about it what tools did you use for editing the pages etc etc Also how did you change the date settings add the paypal and ebay links etc Thanks James Link to comment Share on other sites More sharing options...
di11on Posted September 4, 2006 Author Share Posted September 4, 2006 Thank you so much folks.... that's a very encouraging start. Yup, there are some problems with the Star Product - thanks for pointing it out. The wasted header space is a very good point also, along with the top->catalog thing. How do I fix that by the way? Thanks again folks Link to comment Share on other sites More sharing options...
Jack_mcs Posted September 5, 2006 Share Posted September 5, 2006 Found in searching. Jack Support Links: For Hire: Contact me for anything you need help with for your shop: upgrading, hosting, repairs, code written, etc. All of My Addons Get the latest versions of my addons Recommended SEO Addons Link to comment Share on other sites More sharing options...
tastar Posted September 5, 2006 Share Posted September 5, 2006 re. the breadcrumb redundancy (Top, Catalog), I changed the following (note the comment preceding the code) in catalog/includes/classes/breadcrumb.php: // 3/18/06 changed $i=0 to $i=1 to remove top from the breadcrumb trail function trail($separator = ' - ') { $trail_string = ''; for ($i=1, $n=sizeof($this->_trail); $i<$n; $i++) { Tony Link to comment Share on other sites More sharing options...
di11on Posted September 5, 2006 Author Share Posted September 5, 2006 re. the breadcrumb redundancy (Top, Catalog), I changed the following (note the comment preceding the code) in catalog/includes/classes/breadcrumb.php: // 3/18/06 changed $i=0 to $i=1 to remove top from the breadcrumb trail function trail($separator = ' - ') { $trail_string = ''; for ($i=1, $n=sizeof($this->_trail); $i<$n; $i++) { Tony Thanks a million guys - I've fixed those problems now. Moving the star product box is gonna be a little more tricky, but I definitely think I'll put that on my todo list. Thanks again folks. Jody Link to comment Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.