Here's a little feedback...
Nice work, John! Design is a personal issue so I won't comment so much on the design as the implementation and how it could be improved in terms of efficiency and validity.
So, here's how I would have done the same design (I've removed VBScript for clarity):
Code:
<html>
<head>
<title>SMSG Feedback Form</title>
<style type="text/css">
body
{
background-color: #0000ff;
margin: 0;
padding: 0;
color:white;
font-family: verdana, sans-serif;
}
h1
{
font-family: garamond, sans-serif;
font-size: 24pt;
text-align: center;
}
#menu, #menu2
{
height: 100%;
width: 190px;
background-color: #f0f0f0;
padding: 3px;
border: 1px solid black;
}
#menu
{
float: left;
}
#menu2
{
float: right;
}
form
{
text-align:center;
}
</style>
</head>
<body>
<div id="menu"> </div>
<div id="menu2"> </div>
<h1>Simulator Systems Group's<br>Anonymous Feedback Form</h1>
<form id="myform" method="post" onsubmit="SubmitForm()">
<p><textarea rows="9" cols="50" name="Feedback"></textarea></p>
<p>Feedback is limited to 500 characters in length.</p>
<p><input name="submitter" type="submit" value="Submit Feedback"></p>
<p><input name="submitter" type="reset" value="Reset the Form"></p>
</form>
</body>
</html>
I'll explain my changes as best I can so you can understand my reasoning:
First, it's quicker to use
padding:0 than to use
padding:0px 0px 0px 0px - they both do the same thing. If you specifiy just one parameter then it assumes the same value for all four sides of the box. There's also no need to specify a unit (px, em) for 0 because it's redundant, if you think about it!
Second, always have a default font-family and colour for your body (no need to do HTML as body is content element). Always fall back on a generic font, too. So instead of just having
font-family:veradana have
font-family:verdana, sans-serif. The reason is that if a user doesn't have verdana font installed they will have sans-serif (as it's a generic font included with ever css compatible browser).
Third, you can compact your left and right #menu ids to share the same core attributes like I have done. Then you just specificy that one floats left and the other right. That way you need only edit the core attributes once and they will both share them.
Fourth, it's not valid in HTML to have empty containers, so use a non-breaking space ( ) to put between empty tags, such as your two divs.
Fifth, always remember to close paragraph tags. Whilst not strictly essential in HTML it's a good habit to get into if you move to XHTML.
Sixth, don't use the <CENTER> tag as it's deprecated. Try and do all styling via CSS. See how I've done same by adding text-align:center style to the container FORM tag.
Seventh, I think it's preferable to put Submit button before Reset button as that should be default choice for most situations.
Eigth, it's a good habit to indent all code, HTML, CSS as it makes it more readable (though most programmers are guilty of not doing this!). I personally prefer to align brackets too.
Anyway, good job, John and hope my feedback proves useful
