Okay, perused the site. The following is my personal preference. Please don't take anything the wrong way. You asked for a critique and I shall give it!
First and foremost, your site is...well... big. It's not friendly for those of us on smaller resolution screens (like 800x600 pixels). I would suggest going to
www.statcounter.com and installing a hidden statcounter on your website. It's incredibly easy and the site walks you through the whole process. It'll give you valuable information on how many people visit, how long they stay, how often they visit, along with their ISPs, screen resolutions, and OSs. Not only that, it's absolutely free! I use it on my website and it's incredibly helpful to me to tailor the site around the visitors that come.
I say get rid of the fade-in, fade-out deal. It slows down my browsing and for a button-clicking fool like me I get frustrated. In general, websites should follow the KISS rule unless you're a graphic designer. Keep It Simple Stupid.
I use it on my work all the time. Besides, keeping things simply makes it a whole lot easier on yourself to maintain the website.
I also don't like the green background. I say keep everything the same color. Your bannerhead and your links have a black background. Black is your friend. It matches virtually anything. Also, like others have said, human eyes can't register blue/red, orange/green, or anything like that because the cones for these colors lie next to eachother on our retinas, so our resolution is terrible. I personally like black/orange, black/green, or black/yellow. Black/red is too dark. Human eyes are lazy. They won't make the effort to read something that's difficult to see.
This is long so I'll cut it off here. I can always write more later anyway.