Adding WCF.LoadingOverlayHandler.updateIcon()

diff --git a/wcfsetup/install/files/js/WCF.js b/wcfsetup/install/files/js/WCF.js
index bff2bf2953cc8ef0f0e6ce47db6464b4085e6dd8..bda85578eeba738bc1961f4ff04727db3d50b12a 100755 (executable)
--- a/wcfsetup/install/files/js/WCF.js
+++ b/wcfsetup/install/files/js/WCF.js
@@ -1416,6 +1416,21 @@ WCF.LoadingOverlayHandler = {
                if (this._activeRequests == 0) {
                        this._loadingOverlay.stop(true, true).fadeOut(100);
                }
+       },
+       
+       /**
+        * Updates a icon to/from spinner
+        * 
+        * @param       jQuery  target
+        * @pram        boolean loading
+        */
+       updateIcon: function(target, loading) {
+               var $method = (loading === undefined || loading ? 'addClass' : 'removeClass');
+               
+               target.find('.icon')[$method]('icon-spinner');
+               if (target.hasClass('icon')) {
+                       target[$method]('icon-spinner');
+               }
        }
 };
 
@@ -1724,7 +1739,7 @@ WCF.Action.Delete = Class.extend({
                        WCF.System.Confirmation.show($target.data('confirmMessage'), $.proxy(this._execute, this), { target: $target });
                }
                else {
-                       this._updateIcon($target);
+                       WCF.LoadingOverlayHandler.updateIcon($target);
                        this._sendRequest($target);
                }
        },
@@ -1740,22 +1755,10 @@ WCF.Action.Delete = Class.extend({
                        return;
                }
                
-               this._updateIcon(parameters.target);
+               WCF.LoadingOverlayHandler.updateIcon(parameters.target);
                this._sendRequest(parameters.target);
        },
        
-       /**
-        * Searches for an icon and updates it to a spinner.
-        * 
-        * @param       jQuery  $target
-        */
-       _updateIcon: function($target) {
-               $target.find('.icon').removeClass('icon-remove').addClass('icon-spinner');
-               if ($target.hasClass('icon')) {
-                       $target.removeClass('icon-remove').addClass('icon-spinner');
-               }
-       },
-       
        /**
         * Sends the request
         * 
@@ -1881,6 +1884,7 @@ WCF.Action.Toggle = Class.extend({
                        WCF.System.Confirmation.show($target.data('confirmMessage'), $.proxy(this._execute, this), { target: $target });
                }
                else {
+                       WCF.LoadingOverlayHandler.updateIcon($target);
                        this._sendRequest($target);
                }
        },
@@ -1896,6 +1900,7 @@ WCF.Action.Toggle = Class.extend({
                        return;
                }
                
+               WCF.LoadingOverlayHandler.updateIcon(parameters.target);
                this._sendRequest(parameters.target);
        },
        
@@ -1945,6 +1950,7 @@ WCF.Action.Toggle = Class.extend({
         */
        _toggleButton: function($container, $toggleButton) {
                // toggle icon source
+               WCF.LoadingOverlayHandler.updateIcon($toggleButton, false);
                if ($toggleButton.hasClass('icon-off')) {
                        $toggleButton.removeClass('icon-off').addClass('icon-circle-blank');
                        $newTitle = ($toggleButton.data('enableTitle') ? $toggleButton.data('enableTitle') : WCF.Language.get('wcf.global.button.enable'));