This logic appears to be backwards:
// The below statement checks if the submit button has been clicked and if not displays the form
if (!isset($submit)) {
// The following mail() function is the handler to send off the user input to your e-mail address.mail( "mwood_2k2@msn.com", "Feedback", $name, $message, "From: $email" );
}
I think you want to run the mail() function once the form has been submitted. If your form has been submitted, than $submit will be set.
This statement should never evaluate as true:
else if ($submit == 1)
because $submit will never have an integer value of 1, which is what that if() statement is checking.
Additionally, <center> is a deprecated HTML tag. Though most browsers will respect it, as of the HTML 4.0 spec by per W3C, the preferred solution is to use a stylesheet directive, or, you could put an align="center" attribute in your <table> tag and get the exact same effect.
If you would like to assure that this script works well on many servers, you should seriously consider getting the variables used within the script from either the $_POST array (on PHP 4.2.x and up) or the $HTTP_POST_VARS array (early versions). The growing trend (as well as PHP recommendation) is to keep the register_globals configuration directive in the 'off' position for security reasons.
You may also want to consider what might happen within your script should someone enter an email address which is not valid...
I have a small, simple function that I use for parsing user-world submitted email addresses, which verifies that a value may be an email address. (It doesn't do any DNS lookup on the network, just regular expression pattern match.)
If you're interested, I'd be happy to post it for you.
Have fun...