Jquery on submit not responding

Question

so I have a form wuth a button. On click i need to use to run some google analytics code (_gap.push). This wasnt working so I have decided to do an alert to do some test. The alert showing the word "outside" works however the alert showing the word "inside" is not working. So that means that it is not going into that function

Js code

    applyRegisterNewUserListener: function() {
        alert ("outside")
        // Form that attempts to register users
        $(document).on('submit', 'form#signup-form', function() {
            alert ("inside"
            _gaq.push(['_trackEvent', 'register', 'account', 'form',, false]);
        })
    },

html

<form method="post" id="#signup-form">
<input type="submit" value="submit">
</form>

I really cannot understand what is the problem. The function is being called too.

Your JS-Code is broken. Please read stackoverflow.com/questions/70579/… too. — Grim
– Grim, Commented Aug 7, 2013 at 9:48

Kevin Bowersox · Accepted Answer · 2013-08-07 10:10:10Z

2

The # sign should not be in the id for the form markup.

<form method="post" id="signup-form">

There is also a syntax error in this section of the code.

$(document).on('submit', '#signup-form', function(event) {
    event.preventDefault(); //do not submit form
    alert ("inside"); // alert is missing );
    _gaq.push(['_trackEvent', 'register', 'account', 
                     'form',"", false]); //replace undefined param
});   //add ;

I'm not sure this code needs to be in listener function. It could simply be included as a regular script since it will not execute until the submit event is fired.

edited Aug 7, 2013 at 10:10

answered Aug 7, 2013 at 9:49

Kevin Bowersox

94.6k19 gold badges164 silver badges198 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Rene Zammit Over a year ago

thanks for your help. However when i click on the button, no alert will show. still not working.

Grim Over a year ago

,/*undefined param*/, will produce ,,, that is bad right? May you use ,undefined, instead.

Kevin Bowersox Over a year ago

@PeterRader I was just pointing that out. ,"", would also work.

Kevin Bowersox Over a year ago

@PeterRader Thanks for pointing that out, I added it and moved the comment.

Collectives™ on Stack Overflow

Jquery on submit not responding

1 Answer 1

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related