|
|
#1 |
|
New Member
![]() Join Date: Jan 2012
Posts: 1
|
Hey guys,
Can you review my portfolio site, Think I should get rid of the tables all together and center the content on the header and footer more! Any feedback would be greatly appreciated. Regards Patrick Underneath is the link to the site http://patrickhdoorly.com/ |
|
|
|
|
|
#2 |
|
Platinum Member
![]() |
While you're at it, maybe you could lose the image map as well.
|
|
|
|
|
|
#3 |
|
Bronze Member
![]() Join Date: Jan 2012
Location: Chicago
Posts: 63
|
I think it's strange you use tables instead of div, and you header and footer seem a bit strange, but other than that, your website is incredible. Terrific pictures and designs all around. I am new, but still, I think I can tell good designing (maybe not layout the best ) when I see it.
Do you use javascript to make the slideshow and email form? |
|
|
|
|
|
#4 |
|
Diamond Member
![]() Join Date: Feb 2011
Location: Newcastle, Australia
Posts: 1,137
|
Design is not bad but the code is. Tables and image maps are poor techniques.
Flies completely in the face of your copy. Either change your copy, or change your code. As it stands its bordering on fraudulent.
__________________
if (headhurts == "possibly") { alert ("keep going!"); } else if (headhurts == "yes") { alert ("go to sleep"); } else if (headhurts == "damn !@#$ mofo scripts...") { alert ("give up and have a beer!"); } else { alert ("watch TV"); } |
|
|
|
|
|
#5 |
|
Silver Member
![]() Join Date: Aug 2011
Posts: 187
|
What do you mean bordering on fraudulent? Is it because of the statement about staying up to date with the trends of the industry?
__________________
Check us out: Codedoor Web Design Also looking for people to help write for The Web Design Blog Things should be made as simple as possible, but not any simpler. -Albert Einstein |
|
|
|
|
|
#6 |
|
Diamond Member
![]() Join Date: Feb 2011
Location: Newcastle, Australia
Posts: 1,137
|
Thats exactly what i mean ben. You cant say you “offer“ something, if what you “offer“ is not that at all...
__________________
if (headhurts == "possibly") { alert ("keep going!"); } else if (headhurts == "yes") { alert ("go to sleep"); } else if (headhurts == "damn !@#$ mofo scripts...") { alert ("give up and have a beer!"); } else { alert ("watch TV"); } |
|
|
|
![]() |
| Thread Tools | |
| Display Modes | |
|
|