[snip] I now have: <p><label for="firstname"><span class="red">*</span> First Name: </label><br /> <input class="<?PHP if ($error_msg) {echo "error";} else {echo "normal";}?>" id="firstname" name="firstname" type="text" value="<?php echo $_POST["firstname";]; ?>"> </p> You added one too many, change $_POST["firstname";]; to $_POST['firstname']; <input class="<?PHP if ($error_msg) {echo "error";} else {echo "normal";}?>" id="firstname" name="firstname" type="text" value="<?php echo $_POST['firstname']; ?>"> -- PHP General Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php