Merge branch 'fix-bug-causing-comment-to-submit-twice' into 'master'

Fix bug causing comment form in issue to submit twice when CTRL+Enter is pressed twice See merge request !3655

Merge branch 'fix-bug-causing-comment-to-submit-twice' into 'master'
eebd7533 · Jacob Schatz · 34d7619a · de4d98fd · eebd7533
Commit eebd7533 authored Apr 13, 2016 by Jacob Schatz
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

quick_submit.js.coffee app/assets/javascripts/behaviors/quick_submit.js.coffee +5 -1

No files found.
--- a/app/assets/javascripts/behaviors/quick_submit.js.coffee
+++ b/app/assets/javascripts/behaviors/quick_submit.js.coffee
@@ -29,7 +29,11 @@ $(document).on 'keydown.quick_submit', '.js-quick-submit', (e) ->
  e.preventDefault()

  $form = $(e.target).closest('form')
-  $form.find('input[type=submit], button[type=submit]').disable()
+  $submit_button = $form.find('input[type=submit], button[type=submit]')
+
+  return if $submit_button.attr('disabled')
+
+  $submit_button.disable()
  $form.submit()

 # If the user tabs to a submit button on a `js-quick-submit` form, display a