prevent double note submit

gitlabhq
1 parent 74a89d9e
Showing 3 changed files with 13 additions and 1 deletions Show diff stats
app/views/notes/_form.html.haml
app/views/notes/_notes.html.haml
app/views/notes/create.js.haml
@@ -25,4 +25,4 @@
  
     .clear
     %br
-    = f.submit 'Add note', :class => "lbutton vm"
+    = f.submit 'Add note', :class => "lbutton vm", :id => "submit_note"
@@ -9,6 +9,15 @@
   $('.delete-note').live('ajax:success', function() {  
     $(this).closest('li').fadeOut(); });  
  
+  $("#new_note").live("ajax:before", function(){
+    $("#submit_note").attr("disabled", "disabled"); 
+  })
+
+  $("#new_note").live("ajax:complete", function(){
+    $("#submit_note").removeAttr("disabled"); 
+  })
+  
+
 - if ["issues", "projects"].include?(controller.controller_name)
   :javascript
     $(function(){
@@ -6,3 +6,6 @@
 - else
   :plain
     $("#new_note").replaceWith("#{escape_javascript(render('form'))}");
+
+:plain 
+  $("#submit_note").removeAttr("disabled");
...	...	@@ -25,4 +25,4 @@
25	25
26	26	.clear
27	27	%br
28		- = f.submit 'Add note', :class => "lbutton vm"
	28	+ = f.submit 'Add note', :class => "lbutton vm", :id => "submit_note"
...	...
...	...	@@ -9,6 +9,15 @@
9	9	$('.delete-note').live('ajax:success', function() {
10	10	$(this).closest('li').fadeOut(); });
11	11
	12	+ $("#new_note").live("ajax:before", function(){
	13	+ $("#submit_note").attr("disabled", "disabled");
	14	+ })
	15	+
	16	+ $("#new_note").live("ajax:complete", function(){
	17	+ $("#submit_note").removeAttr("disabled");
	18	+ })
	19	+
	20	+
12	21	- if ["issues", "projects"].include?(controller.controller_name)
13	22	:javascript
14	23	$(function(){
...	...
...	...	@@ -6,3 +6,6 @@
6	6	- else
7	7	:plain
8	8	$("#new_note").replaceWith("#{escape_javascript(render('form'))}");
	9	+
	10	+:plain
	11	+ $("#submit_note").removeAttr("disabled");
...	...