shalendar 0 Posted March 9, 2008 Report 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 Quote Link to post Share on other sites
vchris 3 Posted March 11, 2008 Report 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. Quote Link to post Share on other sites
shalendar 0 Posted March 13, 2008 Author Report 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. Quote Link to post Share on other sites
Synook 47 Posted March 14, 2008 Report 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 Quote Link to post Share on other sites
shalendar 0 Posted March 14, 2008 Author Report 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. Quote Link to post Share on other sites
Synook 47 Posted March 15, 2008 Report Share Posted March 15, 2008 Make it lighter, and make the text grey / black. Or move it out of the main content bit. Quote Link to post Share on other sites
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.