[snip]
Valid HTML requires the type attribute.
function ShowTextBox(){
Text_b_1 = document.getElementById("TextBoxOne");
You should declare that variable using the var keyword to prevent it from
becoming global. It's also preferable to access form controls through the
containing form (if it exists) as that is likely to work across more
browsers.
var elem = document.forms['formName'].elements['controlName'];
Text_b_1.style.display="block"
The style element should be tested before it's used to prevent unnecessary
errors. Furthermore, INPUT is an inline-, not block-level, element.
if(elem.style) {
elem.style.display = 'inline';
}
However, as the control should be visible by default (see below), simply
assigning an empty string ('') will be sufficient.
[snip]
<input type="text" name="textbox1" value="" id="TextBox1"
style="display:none">
As Martin said in his post, that is a bad idea. If scripting is not
available, or the browser doesn't support the manipulation of inline
styles, the page becomes unusable. The textbox should only be hidden by a
script, as only a script can show it again.
[snip]
Mike
