Effect.toggle to return effect ( to be able to do Effect.toggle(eleme…

…nt, 'appear', {sync: true}); )
madrobby · Oct 8, 2009 · eb7c15e · eb7c15e
1 parent ad473ff
commit eb7c15e
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/src/effects.js b/src/effects.js
@@ -145,14 +145,13 @@ var Effect = {
     'blind':  ['BlindDown','BlindUp'],
     'appear': ['Appear','Fade']
   },
-  toggle: function(element, effect) {
+  toggle: function(element, effect, options) {
     element = $(element);
-    effect = (effect || 'appear').toLowerCase();
-    var options = Object.extend({
+    effect  = (effect || 'appear').toLowerCase();
+
+    return Effect[ Effect.PAIRS[ effect ][ element.visible() ? 1 : 0 ] ](element, Object.extend({
       queue: { position:'end', scope:(element.id || 'global'), limit: 1 }
-    }, arguments[2] || { });
-    Effect[element.visible() ?
-      Effect.PAIRS[effect][1] : Effect.PAIRS[effect][0]](element, options);
+    }, options || {}));
   }
 };