I'd go for a completely different colour scheme, bright pastel colours are a lot more pleasant to look at in comparison with those dark yellows that TBH look a bit too much like vomit. Try to make text black (or v.close) with a white background (or v.close), it'll make it easier for people to read especially those with less-than-perfect eyesight.
Avoid centering large amounts of text like that, it's fine for headings and stuff but when it's overused it looks tacky and unproffesional.
You're form points to http://......... so it's no suprise that it doesn't work, you'll need it pointing to a functioning form handler.
I'd distinguish you're external links from internal ones in your navigation and I wouldn't have them opening in frames like that either. It gives the impression that you are snatching content from other sites and I know that's not the impression you want to give.
From a technical point of view: