Site Check for my new portfolio site.

Good colours

Posted: Fri, 2008-07-04 05:50

I like the dark theme, one thing though is maybe some pics to lighten the mood, especially in amongst the text an the services and about me page, but maybe that's just me. Cool navigation too!

Your header images should be

Posted: Fri, 2008-07-04 21:48

1. Your header images should be wrapped in heading tags, not generic spans and divs. In fact, it seems like you haven't used any heading tags at all.
2. From an accessibility point of view, links with the same anchor text that point to different locations ('learn more') are a no-no. You should expand the anchor text to be 'learn more about me' or if you really think it doesn't go with the design, include the 'about me' bit in a hidden span (using the offscreen method).
3. The download your resume doesn't actually download, it loads the PDF in the same window - you should let visitors know that's going to happen (ie tell them what the file is and how large it is).
4. The contrast of the dark blue headers, links and form labels on the grey background is a bit low (ie how to read for some).
5. Don't use tabindex on your form; it's not really needed for such a simple form where the tab order would be that you'd normally follow if it wasn't set (if not used correctly tabindex can often cause more problems than it solves).