Prevent multiple requests from 'done re-ordering'

refinery · Aug 8, 2010 · fa6fc37 · fa6fc37
1 parent 706872a
commit fa6fc37
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/public/javascripts/refinery/admin.js b/public/javascripts/refinery/admin.js
@@ -826,8 +826,11 @@ var list_reorder = {
   }
 
   , disable_reordering: function(e) {
+    if($('#reorder_action_done').hasClass('loading')){
+      return false;
+    }
     if(e) { e.preventDefault(); }
-
+    $('#reorder_action_done').addClass('loading');
     if (list_reorder.update_url != null) {
       serialized = "";
       list_reorder.sortable_list.find('> li[id]').each(function(index, li) {
@@ -877,7 +880,7 @@ var list_reorder = {
 
     $('#site_bar, #header > *:not(script)').fadeTo(250, 1);
     $('#actions *:not("#reorder_action_done, #reorder_action")').not($('#reorder_action_done').parents('li, ul')).fadeTo(250, 1, function() {
-      $('#reorder_action_done').hide();
+      $('#reorder_action_done').hide().removeClass('loading');
       $('#reorder_action').show();
     });
   }

diff --git a/public/stylesheets/refinery/refinery.css b/public/stylesheets/refinery/refinery.css
@@ -1045,6 +1045,9 @@ a img {
 .reorder_icon {
    background-image: url('/images/refinery/icons/arrow_switch.png');
 }
+.reorder_icon.loading{
+  background-image:url('/images/refinery/icons/ajax-loader.gif');
+}
 .user_comment_icon {
   background-image: url('/images/refinery/icons/user_comment.png');
 }