Jump to content

KalijemDesign.net


Recommended Posts

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 post
Share on other sites

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 post
Share on other sites

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 post
Share on other sites

Make it lighter, and make the text grey / black. Or move it out of the main content bit.

Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
×
×
  • Create New...