ueon Posted May 2, 2009 Share Posted May 2, 2009 http://charmsky.ca/index.php Quote Link to comment https://forums.phpfreaks.com/topic/156477-mens-clothing-site/ Share on other sites More sharing options...
Axeia Posted May 3, 2009 Share Posted May 3, 2009 You got an 'align: left' in the CSS (which doesn't excist and causes a warning in the firefox error console) Your images can get compressed way better according to smush.it Also you're using divs instead of images, ruling out the use of an alt text.. and there's text on those images, search engines can't read make any sense of it like that. (I'd use normal text positioned over the images in this case) You're not using a doctype http://charmsky.ca/index.php?view=catalog&catalog=vests I'd use a list for that (<ul>) instead of divs that don't carry any meaning, or tbh a table would do great there, very little information about the product there.. no labels used, not going to do well with search engines. Might wanna optimize the urls for search engines as well. You don't have a favicon, so your site isn't regonisable when in a tab, and the default favicon lookup will end up in a 404 (which according to yahoo's best practices slows down the site more than a small favicon) <h2 id="terms"> <img alt="" src="images/terms_head/terms.jpg"/> </h2> Well that's nice, you used a h2 which is good for headings.. what isn't good is that you did not provide an alt text while it clear has the text "Terms & Conditions" on it. Search engines wont see it and neither will people that make use of browsers that don't render images. Site looks strange being left aligned instead of center, and it's got room left on 1024x768 but it's too big for 800x600. I'd use up at least all space on 1024x768. Quote Link to comment https://forums.phpfreaks.com/topic/156477-mens-clothing-site/#findComment-824617 Share on other sites More sharing options...
perrij3 Posted May 3, 2009 Share Posted May 3, 2009 I would agree that the site looks a little strange being aligned to the left. I have a 21" monitor, so that leaves a lot of space to the right. I also would add/change the hover coding in your CSS so that when someone moves their mouse over a link on your page, the background, font color, etc.. changes to show that it's a link. The other think I think is missing is there is no footer on your page. This is a great place for a legal information link, copyright notification, etc.. I think the design is nice and with a few tweaks, the site will look great. Quote Link to comment https://forums.phpfreaks.com/topic/156477-mens-clothing-site/#findComment-825213 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.