Website Review and Suggestions

**chnceg** · 06-04-2009, 06:04 AM

http://www.wedesignrs.com/

I'm just trying to create some new things for my portfolio since I lost every thing on my old hard drive and don't have any back ups. But anyway, please tell me what you think of the design! This is not a permanent site nor what I'm using the domain for. I would like your critiques, though. Thanks a lot!

**Kenneth** · 06-04-2009, 07:02 AM

I think it looks nice, but something is missing.
Try adding a bigger backgound image or something. The header one is really nice in my opinion.

**delvec** · 06-04-2009, 03:37 PM

Looks very clean-
I wouldnt add anything

**davemies** · 06-04-2009, 04:15 PM

ditto the clean and nice

increasing text 2 notches breaks the layout (it runs behind the blue banner)
you have a few (only a few) coding errors but including a messed up doctype

**chnceg** · 06-04-2009, 05:02 PM

Thanks for the comments every one!

Quote:

Originally Posted by davemies

ditto the clean and nice

increasing text 2 notches breaks the layout (it runs behind the blue banner)
you have a few (only a few) coding errors but including a messed up doctype

I knew I did something wrong with the doctype. I've never written for XHTML before even though I've known what you have to do (code-wise). I have to name it .xhtml, though, right? As for the breaking banner, how would I get to stay fluid with 960.gs and still keep the banner going across the width at 100%?

**davemies** · 06-04-2009, 11:46 PM

no you don't call it xhtml, just plain html, or asp or php or whatever you're using but the dtd should look like this and just this
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

**veikoh** · 06-05-2009, 08:23 AM

Actually by the W3C recommendations you should serve it as application/xhtml instead of text/html
So, it could be better to create new mimetype for XHTML pages.