Collegeboox Posted December 19, 2011 Share Posted December 19, 2011 www.farleycrossflags.com Let me know what you guys like and what should be changed... Link to comment https://forums.phpfreaks.com/topic/253449-critique/ Share on other sites More sharing options...
scootstah Posted December 19, 2011 Share Posted December 19, 2011 It looks like it's from 1995. You used table-based design, not using CSS for images and not using CSS sprites for images. Link to comment https://forums.phpfreaks.com/topic/253449-critique/#findComment-1299147 Share on other sites More sharing options...
floridaflatlander Posted December 19, 2011 Share Posted December 19, 2011 You didn't have a doc type on a page I looked at. If cost is an issue put the whole thing in a wrapper and center, change your hover color in your menu, on the white background it's hard to see and it really is old school but you need to design for your customers and not us. What is the customer expecting? Link to comment https://forums.phpfreaks.com/topic/253449-critique/#findComment-1299267 Share on other sites More sharing options...
KevinM1 Posted December 19, 2011 Share Posted December 19, 2011 It looks like it's from 1995. You used table-based design, not using CSS for images and not using CSS sprites for images. This. Tables should only be used for tabular data, not for layout. You also need a doctype to ensure that the site looks correct in all browsers. A simple: <!doctype html> would suffice. Also, as was said above, you should use sprites for your images. Google 'CSS image sprites' for tutorials on that. I'm not a fan of colored backgrounds with text. Unless it's a white or black background, I find it to be distracting. The animated .gif of the American flag looks hokey. That kind of thing hasn't been in style for ~7 years or so. A static image of a red, white, and blue ribbon would suffice. For your 'Flags' page, why are some larger than others? And why do some have captions while others do not? Link to comment https://forums.phpfreaks.com/topic/253449-critique/#findComment-1299307 Share on other sites More sharing options...
sKunKbad Posted December 20, 2011 Share Posted December 20, 2011 I think you ought to take a look at free template sites and learn to use CSS without a table based layout. It's not the worst site I've ever seen, but it's certainly not going to wow anyone, and that's not good if the person is a potential customer. Link to comment https://forums.phpfreaks.com/topic/253449-critique/#findComment-1299591 Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.