-
Hi,
Good effort.
These are the things I would change:
- Again, add a bit of colour. I know you're going for grayscale but I think that only works for small images, when you've got an image as big as that, it needs some colour.
- I would remove the home button next to the main heading, you've already got a home link in the main nav.
- I would increase font size of the main heading and position it above the main image. Users will want to know what the page is about within a couple seconds and having to look down the page to find the title just increases time on the page.
- No search function? Especially important on an ecommerce site. I notice you have 'Products' in the main nav. I'm guessing that it's just a generic template but you'd need to make HUGE improvements if you're going to be marketing it as an ecommerce site.
- Have you got the logo clickable? You might even be able to remove the home link in the nav if you have.
- In FF 3.6, your 2nd child on the main nav is mis-aligned by about 1em to the bottom. I would even suggest trying to remove the 2nd child as there's more room for error with the user moving their mouse outside of the nav.
Anyway, I'll have a proper look when I've got a bit more time.
Thanks,
Steve

Good review Steve! Welcome to CF!