JANDSCLOTHINGCO Posted August 1, 2013 Share Posted August 1, 2013 Been working and fine tuning the site and modules to suit my needs. Let me know if you notice any errors or issues while browsing the site. Good, bad, or ugly feedback welcome. http://www.jandsclothingco.com Link to comment Share on other sites More sharing options...
Sharak Posted August 1, 2013 Share Posted August 1, 2013 First of all, in newsletter popup, change the code for email input field: instead value="your e-mail address" you should use placeholder="your e-mail address". This way client won't have to delete "your e-mail address" every time. Besides personally I find this kind of popups quite annoying. I don't think your clients want to register for newsletter every time they get back to you. Unless it's only for the time of promotion. I think you could show more products on homepage than just 4 or at least use autoscroll for this slider. My alerts in footer is the only one with an icon. I don't think it was meant that way. Other than above, the store looks fine. Link to comment Share on other sites More sharing options...
vekia Posted August 2, 2013 Share Posted August 2, 2013 may i know what module you're using for "featured products" slider? looks really great! Link to comment Share on other sites More sharing options...
stottycabanas Posted August 2, 2013 Share Posted August 2, 2013 The logo and content give the site a reasonably upmarket feel and there are one or two nice touches, such as the shadowing on the top menu. The featured products slider is good, but I wouldn't have it autoscrolling as suggested above - with the image slider above it, you'd start going cross-eyed! Talking of the image slider, there is a thin banner towards the bottom which overlaps the slider - but maybe that was intended? I feel the black colour scheme is a bit stark, and a few rounded corners wouldn't go amiss, but generally a nice site. Link to comment Share on other sites More sharing options...
JANDSCLOTHINGCO Posted August 2, 2013 Author Share Posted August 2, 2013 @vekia: I am using the free LOF HOME FEATURED module however I modified the css and tpl files to change the appearance of it. @stottycabanas: The banner that is overlapping is intended. Instead of the usual number pager, I wanted the description of the slides Link to comment Share on other sites More sharing options...
stottycabanas Posted August 2, 2013 Share Posted August 2, 2013 OK, understood now. Normally, the number of the image you're displaying is highlighted - if you did that with your text bar (tweak the background colour perhaps), I think that would aid understanding and look pretty good too! Link to comment Share on other sites More sharing options...
JANDSCLOTHINGCO Posted August 2, 2013 Author Share Posted August 2, 2013 Yeah I am trying to figure out how I can do that since technically the bar is a background image I made in photoshop since I couldnt get the pager to display descriptions. I guess a second image file with the alternate color and some kind of coding to change the image would be needed @sharak: Once someone hits the subscribe button that window won't popup again either Link to comment Share on other sites More sharing options...
JANDSCLOTHINGCO Posted August 2, 2013 Author Share Posted August 2, 2013 Ok I came up with somewhat of a solution for the slider pager to show which slide you are on. Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now