onClick doesn't fire with only one input when pressing enter

Question

I'm having trouble with events in Internet Explorer 7.

When I have a form with two or more input[type=text] and I press enter, the events occurs in this order:

submit button (onClick)
form (onSubmit)

Sample code:

<form onSubmit="{alert('form::onSubmit'); return false;}">
    <input type="text">
    <input type="text">
    <input type="submit" onClick="{alert('button::onClick');}">
</form>

If I have only one input[type=text] and I press enter the submit button onClick event doesn't fire. Sample code:

<form onSubmit="{alert('form::onSubmit'); return false;}">
    <input type="text">
    <input type="submit" onClick="{alert('button::onClick');}">
</form>

This question addresses this issue: [ENTER key on a FORM with a single Input Field, will automatically SUBMIT with GET](http://stackoverflow.com/questions/1370021/enter-key-on-a-form-with-a-single-input-field-will-automatically-submit-with-ge) — lawnsea, Jul 07 '10 at 15:26

score 5 · Answer 1 · answered Aug 13 '08 at 00:08

The button's onclick should (I think) only fire if the button is actually clicked (or when the focus is on it and the user clicks enter), unless you've added logic to change that.

Is the addition of the extra textbox possibly changing the tab order of your elements (perhaps making the button the default control in that case)?

mplungjan · Answer 2 · 2010-10-04T13:45:15.837

3

From the dawn of browsers, a single input field in a form would submit on enter with or without a submit button. This was to make it simpler for people to search a site from a single search field.

If you need to execute some javascript in a form, it is safer practice to use the onsubmit of the form (and return false to stop submission) rather than executing script in the onClick of the submit button. If you need to execute javascript from a button, use type="button" instead of type="submit" - hope this clarified what I meant

edited Oct 04 '10 at 13:45

answered Jul 07 '10 at 15:19

mplungjan

169,008
28
173
236

I didn't downvote, but I'm having trouble parsing the last sentence. ("never do anything onClick of a submit button but always do it in ... a button onClick"???) – Tim Pietzcker Oct 04 '10 at 12:24

score 1 · Answer 3 · answered Aug 22 '08 at 16:09

Interestingly, if you click on the screen (remove the focus from the textbox) on second example with only one textbox, the event onClick fires... So it's not an expected behaviour since it only occurs when you have just one textbox and you have the focus on the textbox.
I'm afraid you've found a bug on the browser and you'll have to find a workaround, or avoid using the onClick event in that case.
I use the onSubmit event for validations because it's a "safer" event that is more likely to work on different browsers and situations.

score 1 · Answer 4 · answered Aug 22 '08 at 16:13

1

Out of curiosity, are you using a DOCTYPE, and if so, which one? I'm not saying incompatabilities with the DOCTYPE are the issue, but quirks mode is something to rule out before trying anything else.

answered Aug 22 '08 at 16:13

Brian Warshaw

22,657
9
53
72

score 1 · Answer 5 · answered Aug 12 '08 at 23:43

1

If you want code to run when the user presses enter, just use the onSubmit handler.

If you want code to run when the user presses the button, and not when the user presses enter, use a button other than type="submit".

answered Aug 12 '08 at 23:43

andrewrk

30,272
27
92
113

score 0 · Answer 6 · answered Mar 03 '16 at 14:52

You might want to include a dummy hidden input element to recreate the situation where you had two input elements... that way, you'll get both of the events fired

<FORM onSubmit="{alert('form::onSubmit'); return false;}">
    <INPUT TYPE="text">
    <input type="hidden" name="dummy">
    <INPUT TYPE="submit" onClick="{alert('buttom::onClick');}">
</FORM>

IE has a lot of confusing fixes that needs to be done for improving compatibility of our code

onClick doesn't fire with only one input when pressing enter

6 Answers6