|
|
|
|
![]() ![]() |
May 22 2005, 07:25 PM
Post
#1
|
|
|
Super Member ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Group: Members Posts: 362 Joined: 2-March 05 From: The Netherlands Member No.: 4,097 |
Since i had my trap17 account i think i have been working on a dozen different sites...and guess what all unfinished..lol
Anyway, to keep track of my little projects i started (and will be finishing!) my portfolio site I have created everything myself, no copied material Anyway, I would like to have some feedback http://www.online.trap17.com/home.php (i might make it my portal page for all the other sites) By The way, does anyone agree with me that the "content" (where all the content will be) should have a greyish background? I think its too much white, but id like to keep the body white, a change for all the black backgrounds i always use I love all comment, so shoot |
|
|
|
May 22 2005, 07:28 PM
Post
#2
|
|
|
Super Member ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Group: Members Posts: 362 Joined: 2-March 05 From: The Netherlands Member No.: 4,097 |
Err, i clicked the link and the navigation seems to be screwed up the first time you visit the page -.- damn frame scrollbars!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
|
|
|
|
May 22 2005, 08:01 PM
Post
#3
|
|
|
Privileged Member ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Group: Members Posts: 661 Joined: 10-January 05 Member No.: 3,189 |
Please refrain from using iframes, to avoid counding cliche, they are soo 1990's. The header also needs some spicing up as well.
|
|
|
|
May 22 2005, 10:05 PM
Post
#4
|
|
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Group: Members Posts: 1,161 Joined: 9-May 05 From: Brisbane, QLD Member No.: 6,818 |
Not too much there to comment on at the moment, but like whyme says, your header seems a bit pointless - it doesn't say anything so you have no idea what the site's called. Also I think your nav buttons are too small & hard to read. And I don't think you really need to set the onmousedown and onmouseup values for those links cos they're the same as onmouseover and onmouseout.
|
|
|
|
May 22 2005, 10:27 PM
Post
#5
|
|
|
Super Member ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Group: Members Posts: 362 Joined: 2-March 05 From: The Netherlands Member No.: 4,097 |
I uploaded a new header wich does tell a bit more about my portfolio site (the animation of my projects, Java, PHP, and more.....lol)
And should a portfolio site have a name? I only set onmouseover values, and they should turn reddish..they do for me Took out the iframe for the navigation And are the buttons not readable? I think the visitor doesnt have a hard time to read Home|Projects|About me|Contact do they? Anyone else got problem with the font size? i can adjust em ^^..if needed (I kinda like them this size |
|
|
|
May 23 2005, 01:00 AM
Post
#6
|
|
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Group: Members Posts: 1,161 Joined: 9-May 05 From: Brisbane, QLD Member No.: 6,818 |
QUOTE(HmmZ @ May 23 2005, 08:27 AM) And should a portfolio site have a name? If it's a portfolio, I think it should have the name of the person in there at least somewhere. You shouldn't have to go to the About Me page to find out whose portfolio you're looking at. It should at least be in the title tag. QUOTE(HmmZ @ May 23 2005, 08:27 AM) I only set onmouseover values, and they should turn reddish..they do for me What's all this then? CODE <a href="http://www.online.trap17.com/content.php" target="content" onmouseover="changeImages('nav_Home', 'images/nav_Home-over.gif'); return true;" onmouseout="changeImages('nav_Home', 'images/nav_Home.gif'); return true;" onmousedown="changeImages('nav_Home', 'images/nav_Home-over.gif'); return true;" onmouseup="changeImages('nav_Home', 'images/nav_Home-over.gif'); return true;"> Looks like 2 more lines of code than is really necessary to me. QUOTE(HmmZ @ May 23 2005, 08:27 AM) And are the buttons not readable? I think the visitor doesnt have a hard time to read Home|Projects|About me|Contact do they? Anyone else got problem with the font size? i can adjust em ^^..if needed (I kinda like them this size sad.gif ) For me, personally, they're too small especially with that thin font and dark colour. It's not that they're unreadable, but as a designer, when I look at your page, one of the first things I thought was: "nav buttons are too small". You have to search the page to find them and that's without any content on there. You should be making your site's navigation an easy matter for your users. |
|
|
|
May 23 2005, 03:22 AM
Post
#7
|
|
|
Newbie [Level 2] ![]() ![]() Group: Members Posts: 38 Joined: 19-May 05 Member No.: 7,250 |
QUOTE(HmmZ @ May 23 2005, 03:25 AM) By The way, does anyone agree with me that the "content" (where all the content will be) should have a greyish background? I think its too much white, but id like to keep the body white, a change for all the black backgrounds i always use I don't agree that the content should have a greyish background. Although your idea of providing contrast between the the content background and site background is great. Maybe it should be the other way around. Try #f7f7f7 for your site background and keep your content background white. You can also try #cccccc for your the site background. Either of the two, #f7f7f7 and #cccccc, combine great with white. Then if you want you can add a shadow to one or both sides. Nice Site Anyway. Where did you create the banner? |
|
|
|
May 23 2005, 07:18 AM
Post
#8
|
|
|
Privileged Member ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Group: Members Posts: 821 Joined: 6-March 05 Member No.: 4,202 |
I agree with that. Change the background of the site, not the colour in which the images are loaded. I would try to make the menu more visible, and enlarge the zone of the mouseover.
Very stylish anyway. |
|
|
|
May 23 2005, 10:03 AM
Post
#9
|
|
|
Super Member ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Group: Members Posts: 362 Joined: 2-March 05 From: The Netherlands Member No.: 4,097 |
CODE <a href="http://www.online.trap17.com/content.php" target="content" onmouseover="changeImages('nav_Home', 'images/nav_Home-over.gif'); return true;" onmouseout="changeImages('nav_Home', 'images/nav_Home.gif'); return true;" onmousedown="changeImages('nav_Home', 'images/nav_Home-over.gif'); return true;" onmouseup="changeImages('nav_Home', 'images/nav_Home-over.gif'); return true;"> Thats the generated code of Imageready rollover slices, but ill see what i can do wit |