jame
jame

Reputation: 132

jQuery submit function not working properly

I have some questions with jQuery submit function.

here is work environment

jQuery: 1.7.2, chrome (18.0.1025.168 m).

There are 2 problems.

1st:

my codes like this

html

<form id="test" action="..." method="post">
     <input type="text" />
     <input type="submit">
</form>

jQuery

$('#test').submit(function(){
      return false;
})

the problem is it works fine in firefox and opera but chrome.

2st:

html: as above.

jQuery:

$('#test').submit(function(){
      if(...)
         alert(something..);
      return false;
})

it dosen't work in firefox,opera and chrome. it always trigger form.submit why.

I am very confused it. who can figure it out thanks!

Upvotes: 2

Views: 21310

Answers (4)

yoel halb
yoel halb

Reputation: 12711

This is sometimes a result of JS libraries or custom code that conflicts with the behavior of the submit handler, (for example having another submit handler that submits explicitly the form).

The ideal solution would be to debug and find the conflicting code and solve it.

But as a workaround one can change to use instead a click handler on the submit button, instead of a submit handler on the form.

One can do something like this:

    <input type="submit" id="btnSubmit">    

Then in code you can do

$('#btnSubmit').click(function(){ //Actually we can do 'input[type="submit"]'
    return false;
})

However one has to be careful if the form allows to be submitted by hitting "enter", as in this case the calling of the click handler might be browser dependent and/or based on the libraries in use, see the comments in onsubmit="return false" has no effect on Internet Explorer 7/8 (form is still submitted) on Dirk Paessler's answer for more info.

But in order to be browser independent one might catch the "enter" key press button as well, as in the following code:

document.onkeypress = function(evt) {
   var evt = (evt) ? evt : ((event) ? event : null);
   var node = (evt.target) ? evt.target : ((evt.srcElement) ? evt.srcElement : null);
   if ((evt.keyCode == 13) && (node.type == "text")) { 
         /*return true or false here based on your logic*/;
   }
}

However one might also try the other suggestions in this thread such as calling e.preventDefault() directly, although this is what Jquery actually does behind the scenes, still it might be that your conflicting code is fine with it

Upvotes: 1

moribvndvs
moribvndvs

Reputation: 42497

Don't use return false;. Try this instead:

$('#test').submit(function(event){
      event.preventDefault();
});

Upvotes: 11

mprabhat
mprabhat

Reputation: 20323

If you just dont want to submit the page you can simple use

onsubmit="return false" or if you want conditional submission that also is possible with the code that you have.

I don't see any issue with this

DEMO

Code:

$('#test').submit(function(event) {
    if(jQuery('#textbox').val() == '') {
       alert('Not submitting');
       //event.preventDefault(); Not necessary but better
       return false;
    }
})

Upvotes: 3

iambriansreed
iambriansreed

Reputation: 22241

Replace:

$('#test').submit(function(){
      return false;
});

with:

$('form#test').submit(function(event){    
      event.preventDefault();
});

Upvotes: 4

Related Questions