r2709 - Effects core: Reorganized method overloading.

r2709 - Effects core: Reorganized method overloading.


Author: scott.gonzalez
Date: Thu Jun 11 14:21:35 2009
New Revision: 2709
Modified:
trunk/ui/effects.core.js
Log:
Effects core: Reorganized method overloading.
Modified: trunk/ui/effects.core.js
==============================================================================
--- trunk/ui/effects.core.js    (original)
+++ trunk/ui/effects.core.js    Thu Jun 11 14:21:35 2009
@@ -155,18 +155,7 @@
    return [effect, options, speed, callback];
}
-//Extend the methods of jQuery
$.fn.extend({
-
-    //Save old methods
-    _show: $.fn.show,
-    _hide: $.fn.hide,
-    __toggle: $.fn.toggle,
-    _addClass: $.fn.addClass,
-    _removeClass: $.fn.removeClass,
-    _toggleClass: $.fn.toggleClass,
-
-    // New effect methods
    effect: function(effect, options, speed, callback) {
        var args = _normalizeArguments.apply(this, arguments),
            // TODO: make effects takes actual parameters instead of a hash
@@ -180,6 +169,7 @@
        return effectMethod && !$.fx.off ? effectMethod.call(this, args2) : this;
    },
+    _show: $.fn.show,
    show: function(speed) {
        if (!speed || typeof speed == 'number' || $.fx.speeds[speed]) {
            return this._show.apply(this, arguments);
@@ -190,6 +180,7 @@
        }
    },
+    _hide: $.fn.hide,
    hide: function(speed) {
        if (!speed || typeof speed == 'number' || $.fx.speeds[speed]) {
            return this._hide.apply(this, arguments);
@@ -200,6 +191,8 @@
        }
    },
+    // jQuery core overloads toggle and create _toggle
+    __toggle: $.fn.toggle,
    toggle: function(speed) {
        if (!speed || typeof speed == 'number' || $.fx.speeds[speed] ||
            typeof speed == 'boolean' || $.isFunction(speed)) {
@@ -211,15 +204,21 @@
        }
    },
+    _addClass: $.fn.addClass,
    addClass: function(classNames, speed, easing, callback) {
        return speed ? $.effects.animateClass.apply(this, [{ add: classNames
},speed,easing,callback]) : this._addClass(classNames);
    },
+
+    _removeClass: $.fn.removeClass,
    removeClass: function(classNames,speed,easing,callback) {
        return speed ? $.effects.animateClass.apply(this, [{ remove: classNames
},speed,easing,callback]) : this._removeClass(classNames);
    },
+
+    _toggleClass: $.fn.toggleClass,
    toggleClass: function(classNames,speed,easing,callback) {
        return ( (typeof speed !== "boolean") && speed ) ?
$.effects.animateClass.apply(this, [{ toggle: classNames
},speed,easing,callback]) : this._toggleClass(classNames, speed);
    },
+
    morph: function(remove,add,speed,easing,callback) {
        return $.effects.animateClass.apply(this, [{ add: add, remove: remove
},speed,easing,callback]);
    },