Preventing the Enter key from submitting a form

[donpro]

Hi,

I have the following function that check for the <Enter> key being
pressed on an <Input> field:

function disableCR(e) {
if (window.event) {
key = window.event.keyCode;
} else {
key = e.which;
}

alert(key);
if (key == 13) {
alert('false');
return false;
} else {
alert('true');
return true;
}
}

My previous HTML was like so:
<input type="text" name="total" value="100" size="6" maxlength="4"
onkeypress="return disableCR(event)">

I would like to convert this into an Event Listener using DOM but not
sure how to do this. Has anyone done this?

 

[Thomas 'PointedEars' Lahn]

I have the following function that check for the <Enter> key being
pressed on an <Input> field:

function disableCR(e) {
if (window.event) {
key = window.event.keyCode;
} else {
key = e.which;
}
[...]
}

My previous HTML was like so:
<input type="text" name="total" value="100" size="6" maxlength="4"

`type="text"' is redundant as that is the default value for that attribute.

onkeypress="return disableCR(event)">

I would like to convert this into an Event Listener using DOM but not
sure how to do this. Has anyone done this?

You are already using a event listener, and you are using the DOM; in fact,
you are supporting at least three DOMs: the MSHTML-DOM with window.event,
the W3C-DOM and the NN4/Gecko-DOM with `e', and the NN4/Gecko-DOM with
`e.which'.

The local `event' property in the execution context of the event listener
code is proprietary to refer to an Event object, indeed, but I have yet to
see a DOM that does not have it. Avoiding it will only lead to your
scripting branches for several DOM Event implementations, of which only
three are known.

IOW: If it ain't broken, don't fix it.


PointedEars

 

[Bart Van der Donck]

`type="text"' is redundant as that is the default value for that attribute.

I would rather advocate the following:

| This is not a required attribute, but we think you should
| include it. If omitted, IE 5.5 will still display a text field,
| but Netscape 4.7 will not.

http://www.w3schools.com/tags/tag_input.asp