Soldato
- Joined
- 12 May 2007
- Posts
- 3,896
- Location
- Bristol
Did you submit it already?
Not bad if this is your first site. Here's some constructive crit.
- Remove the list-style from your main navigation. (it shows in FF)
- Choose some link styles.
- More padding around your h2 (which should actually be an h1 - you should ALWAYS include one and only one h1 on every page)
- Why is the main image inside an h2?
- Add more line-height to your copy. The text looks cramped.
- Those bottom images don't need to link to such large versions. I'd also just drop them in one of the numerous lightbox type popups.
- Crop all your main images so they're the same height across the pages.
- You can certainly style your contact form better.
- Add a nice border to all your images. A nice light color which doesn't stand out. Remove the default border around images which are links.
Not bad if this is your first site. Here's some constructive crit.
- Remove the list-style from your main navigation. (it shows in FF)
- Choose some link styles.
- More padding around your h2 (which should actually be an h1 - you should ALWAYS include one and only one h1 on every page)
- Why is the main image inside an h2?
- Add more line-height to your copy. The text looks cramped.
- Those bottom images don't need to link to such large versions. I'd also just drop them in one of the numerous lightbox type popups.
- Crop all your main images so they're the same height across the pages.
- You can certainly style your contact form better.
- Add a nice border to all your images. A nice light color which doesn't stand out. Remove the default border around images which are links.
