Portfolio Website Crit (XHTML/CSS/PHP/JS)

[aspacecodyssey]

Just looking for some feedback. I do mostly front-end web design (I'm fairly well versed in PHP/SQL, but design is my forté) as well as standard print/branding stuff. I've been going with this for a little while, maybe some people will have some fresh ideas.

 

http://aspacecodyssey.com/

[QuickOldCar]

Impressive, one of the best working and functional sites I've seen in a long time.

 

Good job.

 

Now time to animate your avatar, ha.

[lostprophetpunk]

Not a bad job there. Colours are a bit clashing though. Nice work on the design and function of the website.

 

However there are a couple of things I would like to mention...

<div id="errorie6" class="return hidden">
Uh-oh. Looks like you’re using an outdated, unsecure, or generally web-developer-headache-inducing browser. You’re going to miss out on some features of this website.<br />
<br />
That’s okay. <a href="http://www.getfirefox.com">Get Firefox</a> to solve that problem.
</div>

That is a bit harsh to some users. Part of the users who will view your site will still be using IE6 at the moment. I know it's annoying but as a web developer (in which you mention design is your forté) you should be able to make the site compatible for all browsers. Also, there are other browsers out there other than 'Firefox', such as Google Chrome, Opera and Safari in which users may be interested in.

 

On a lighter note, the typography of the website could be improved. Use a different font for headings and a different font for the main body of text on the page, to make it more appealing to users.

 

Anyway, keep up the good work.

[the182guy]

Don't put raw content directly into divs. This is bad. Put text content and images in <p> tags and don't use <br /> tags to create paragraphs, that's what <p> tags are for.