98

I have a submit button at the end of the form.

I have added the following condition to the submit button:

onClick="this.disabled=true;
this.value='Sending…';
this.form.submit();"

But when it moves to the next page, the parameters did not pass and null values are passed.

2
  • This is probably not the fault of the calls you show. Please show the full HTML of the form.
    – Pekka
    Jul 29, 2010 at 20:52
  • 4
    Don't use the onclick event of the submit button - use the onsubmit event of the form. Otherwise you won't catch a submission from the keyboard.
    – Nick
    Jul 29, 2010 at 21:08

19 Answers 19

142

You should first submit your form and then change the value of your submit:

onClick="this.form.submit(); this.disabled=true; this.value='Sending…'; "
9
  • 6
    This is the best answer. Some browsers don't know to submit the form beforehand, and this solution works around that.
    – Nathanael
    Aug 12, 2014 at 17:02
  • 9
    onclick should be lowercase Feb 5, 2015 at 15:04
  • 1
    Won't work as expected if HTML5 validation is triggered first.
    – cyrotello
    Oct 5, 2017 at 4:39
  • 1
    For me, 'this.disabled=true' prevents back end function from firing, even though it appears after the submit! To get around this I implemented a bool value and if statement to check first or subsequent clicks, and returned false for subsequent.
    – Radderz
    Oct 13, 2017 at 16:50
  • Working with ASP.NET Update Panel I disabled the Submit behavior. 'OnClick="btnSave_Click" OnClientClick="this.disabled=true; this.value='Saving ... Please Wait.';" UseSubmitBehavior="false"' Nov 9, 2017 at 22:07
73

Probably you're submitting the form twice. Remove the this.form.submit() or add return false at the end.

you should end up with onClick="this.disabled=true; this.value='Sending…';"

5
  • 32
    Careful though as this does not work in IE8 and Chrome...well it works in that it does what the script says to do, but a consequence is that it also disables the submitting of the form. Mar 8, 2013 at 8:58
  • 1
    See @andreaskoberle's answer Jan 21, 2016 at 4:59
  • 10
    for button type=submit tags replace this.value with this.innerText Dec 18, 2018 at 11:37
  • @Andreas Köberle answer below is better, especially if using any of the ASP.NET frameworks. Oct 6, 2020 at 23:06
  • its 2022, and this solution works perfectly on Chrome 102.0.5005.115
    – WiiLF
    Jun 15, 2022 at 16:31
38

tested on IE11, FF53, GC58 :

onclick="var e=this;setTimeout(function(){e.disabled=true;},0);return true;"
2
  • 6
    This should be accepted ; others doesn't send the form Jul 24, 2019 at 16:11
  • This works great, but how do I change the "value" when clicked?
    – robert0
    Jul 13, 2021 at 10:29
27

You need to disable the button in the onsubmit event of the <form>:

<form action='/' method='POST' onsubmit='disableButton()'>
    <input name='txt' type='text' required />
    <button id='btn' type='submit'>Post</button>
</form>

<script>
    function disableButton() {
        var btn = document.getElementById('btn');
        btn.disabled = true;
        btn.innerText = 'Posting...'
    }
</script>

Note: this way if you have a form element which has the required attribute will work.

0
10

Disabled HTML forms elements aren't sent along with the post/get values when you submit the form. So if you disable your submit button once clicked and that this submit button have the name attribute set, It will not be sent in the post/get values since the element is now disabled. This is normal behavior.

One of the way to overcome this problem is using hidden form elements.

0
7

the trick is to delayed the button to be disabled, and submit the form you can use window.setTimeout('this.disabled=true',0); yes even with 0 MS is working

1
  • 1
    but what if the user presses the button in that fraction of time ?
    – jospratik
    Sep 26, 2013 at 7:21
4

Using JQuery, you can do this..

$("#submitbutton").click(
   function() {
      alert("Sending...");
      window.location.replace("path to url");
   }
);
1
  • 3
    Please don't use alerts for that, they block the page until clicked Dec 18, 2018 at 11:36
2

If you disable the button, then its name=value pair will indeed not be sent as parameter. But the remnant of the parameters should be sent (as long as their respective input elements and the parent form are not disabled). Likely you're testing the button only or the other input fields or even the form are disabled?

2
function xxxx() {
// submit or validate here , disable after that using below
  document.getElementById('buttonId').disabled = 'disabled';
  document.getElementById('buttonId').disabled = '';
}
2

Here's a drop-in example that expands on Andreas Köberle's solution. It uses jQuery for the event handler and the document ready event, but those could be switched to plain JS:

(function(document, $) {

  $(function() {
    $(document).on('click', '[disable-on-click], .disable-on-click', function() {
      var disableText = this.getAttribute("data-disable-text") || 'Processing...';

      if(this.form) {
        this.form.submit();
      }

      this.disabled = true;

      if(this.tagName === 'BUTTON') {
        this.innerHTML = disableText;
      } else if(this.tagName === 'INPUT') {
        this.value = disableText;
      }
    });
  });

})(document, jQuery);

It can then be used in HTML like this:

<button disable-on-click data-disable-text="Saving...">Click Me</button>
<button class="disable-on-click">Click Me</button>
<input type="submit" disable-on-click value="Click Me" />
1
  • I like this solution because it follows the 'Don't Repeat Yourself' (DRY) principle. Once the first chunk is added you can re-use it simply by using a class.
    – Calabacin
    Apr 14, 2020 at 10:04
1

I don't think you need this.form.submit(). The disabling code should run, then it will pass on the click which will click the form.

2
  • 1
    If you don't put "this.form.submit()" then form will not submit in this case. Aug 31, 2017 at 6:15
  • i am using <button type="button" class="addb btn btn-primary rounded-pill py-2 btn-block" type="submit" data-voice_sku="'.$row["voice_sku"].'" data-voice_name="'.$row["voice_name"].'" onclick="this.disabled=true">Add to Playlist</button> if i refresh the page disable gone, how to solve this?
    – Gem
    Sep 28, 2019 at 10:52
1

Another solution i´ve used is to move the button instead of disabling it. In that case you don´t have those "disable" problems. Finally what you really want is people not to press twice, if the button is not there they can´t do it.

You may also replace it with another button.

0

Your question is confusing and you really should post some code, but this should work:

onClick="this.disabled=true; this.value='Sending...'; submitForm(); return false;"

I think that when you use this.form.submit() it's doing what happens naturally when you click the submit button. If you want same-page submit, you should look into using AJAX in the submitForm() method (above).

Also, returning false at the end of the onClick attribute value suppresses the default event from firing (in this case submitting the form).

0
 
    A better trick, so you don't lose the value of the button is

    function showwait() {
    document.getElementById('WAIT').style['display']='inline';
    document.getElementById('BUTTONS').style['display']='none';
    }
 

wrap code to show in a div

id=WAIT style="display:none"> text to display (end div)

wrap code to hide in a div

id=BUTTONS style="display:inline"> ... buttons or whatever to hide with
onclick="showwait();" (end div)

0

In my case this was needed.

Disable submit button on form submit

It works fine in Internet Explorer and Firefox without it, but it did not work in Google Chrome.

The problem is that you are disabling the button before it can actually trigger the submit event.

1
  • 1
    It's generally good practice to show the appropriate code in your answer, instead of just a link. Jul 23, 2015 at 17:04
0

I think easy way to disable button is :data => { disable_with: "Saving.." } This will submit a form and then make a button disable, Also it won't disable button if you have any validations like required = 'required'.

1
  • 2
    This behavior seems to be Ruby on Rails specific. Dec 18, 2018 at 11:35
0

In this working example, the user confirms in JavaScript that he really wants to abort. If true, the button is disabled to prevent double click and then the code behind which updates the database will run.

<asp:button id="btnAbort" runat="server" OnClick="btnAbort_Click" OnClientClick="if (!abort()) {return false;};" UseSubmitBehavior="false" text="Abort" ></asp:button>

I had issues because .net can change the name of the button

function abort() {
    if (confirm('<asp:Literal runat="server" Text="Do you want to abort?" />')) {
        var btn = document.getElementById('btnAbort');
        btn.disabled = true;
        btn.innerText = 'Aborting...'
        return true;
    }
    else {
        return false;
    }  
}

Because you are overriding the OnClick with OnClientClick, even if your validation method succeeds, the code behind wont work. That's why you set UseSubmitBehavior to false to make it work

PS: You don't need the OnClick if your code is in vb.net!

0

Okay, I did a lot of research on how to make this work perfectly. So the best option is to create a set timeout for disabling a button onclick.

Now, the problem arise when there is a submit function running on the backend. Then the events become stacked in a queue and whenever the javascript "button.disabled == true"is added to the onclick event, only the first action(i.e. disabling the button) gets triggered and not the submit action which is running in the backend (this backend submit function can comprise of anything such as $.ajax).

For disabling Single button on click :

function() {  // I always create anonymous function to avoid polluting global space
    var btn = document.getElementsByClassName("btn");
    btn.onclick = function() {
        setTimeout(function() {
            backButton.disabled = true;
        }, 0);
    };
}
}();

This code will disable your button and also would run the function on the queue. timeout = 0 actually is used for firing subsequent backend tasks.

For disabling all btns in the screen :

(function() {
    let i, element, list, o;
    element = document.getElementsByClassName("classx");
    if (element) {
        element = element[0];
        list = element.getElementsByTagName("button");
        for (i = 0; i < list.length; i++) {
            o = list[i];
            o.onclick = function() {
                setTimeout(function() {
                    let i;
                    for (i = 0; i < list.length; i++) {
                        list[i].disabled = true;
                    }
                }, 0);
                return true;
            }
        }
    }
})();

This would help you disable all of the buttons present in the page (just use it according to your usecase).

Also, this (disabled button) is a good use case for settimeout=0, functionality description as it will "defer" the call until the currently "stacked javascript events" are finished.

Thank you and hope this helps someone's in the future.

-1

I did the trick. When set timeout, it works perfectly and sending all values.

    $(document).ready(function () {
        document.getElementById('btnSendMail').onclick = function () {
            setTimeout(function () {
                document.getElementById('btnSendMail').value = 'Sending…';
                document.getElementById('btnSendMail').disabled = true;
            }, 850);
        }
    });
1
  • people will click several times in 850ms
    – cweiske
    Apr 23, 2021 at 15:01

Not the answer you're looking for? Browse other questions tagged or ask your own question.