shalendar Posted March 9, 2008 Share Posted March 9, 2008 KalijemDesign.netThis is my portfolio site. I know the design isnt that great, but im thinking of redoing it. again. give me your opinions on my site and the sites ive done. Thanks Link to comment Share on other sites More sharing options...
vchris Posted March 11, 2008 Share Posted March 11, 2008 Good work but there's a couple things I think could be better/fixed. Your HTML does not validate. There is 17 errors found. CSS validates. Nice div design. Your top navigation is a bit hard to see for me so imagine older people... I'd increase the contrast for your top nav. Your CNN.com news seems like it's an included tool. Maybe you don't have the knowledge but you could use an rss feed and php, asp or coldfusion to integrate the cnn news in your site. In your content area the blue text is kinda hard to read on the dark gray background.I'd say mainly you need to work on your text and background contrast. If this is improved your site will be much better. Link to comment Share on other sites More sharing options...
shalendar Posted March 13, 2008 Author Share Posted March 13, 2008 hmm. well it validated when i put the links on, but ive done a lot of changing since then. ive considered learning rss, or just removing the feed. the php portfolio is a recent edition, so its not completely polished. i didnt realise the text was hard to read. thanks. Link to comment Share on other sites More sharing options...
Synook Posted March 14, 2008 Share Posted March 14, 2008 RSS is extremely easy to learn. The portfolio box looks out of place, and I can't say your page is exactly valid... besides that and the links, its not bad! Keep developing Link to comment Share on other sites More sharing options...
shalendar Posted March 14, 2008 Author Share Posted March 14, 2008 what should i do with the portfolio box. i recently added that with php, so it doesnt really fit with the design. Link to comment Share on other sites More sharing options...
Synook Posted March 15, 2008 Share Posted March 15, 2008 Make it lighter, and make the text grey / black. Or move it out of the main content bit. Link to comment Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.