]> source.dussan.org Git - jquery-ui.git/commitdiff
clip effect should be animating the wrapper
authorAaron Eisenberger <aaronchi@gmail.com>
Mon, 16 Jun 2008 13:18:48 +0000 (13:18 +0000)
committerAaron Eisenberger <aaronchi@gmail.com>
Mon, 16 Jun 2008 13:18:48 +0000 (13:18 +0000)
ui/effects.clip.js

index d1329f46eb02c58b3e08d66b9e3b9db158652d8e..6bfeddeb3cb5ec58be23a59f15baaeda296e350a 100644 (file)
@@ -17,7 +17,7 @@ $.effects.clip = function(o) {
        return this.queue(function() {
 
                // Create element
-               var el = $(this), props = ['position','top','left','width','height'];
+               var el = $(this), props = ['position','top','left'];
                
                // Set options
                var mode = $.effects.setMode(el, o.options.mode || 'hide'); // Set Mode
@@ -25,13 +25,13 @@ $.effects.clip = function(o) {
                
                // Adjust
                $.effects.save(el, props); el.show(); // Save & Show
-               $.effects.createWrapper(el).css({overflow:'hidden'}); // Create Wrapper
+               var wrapper = $.effects.createWrapper(el).css({overflow:'hidden'}); // Create Wrapper
                var ref = {
                        size: (direction == 'vertical') ? 'height' : 'width',
                        position: (direction == 'vertical') ? 'top' : 'left'
                };
-               var distance = (direction == 'vertical') ? el.height() : el.width();
-               if(mode == 'show') { el.css(ref.size, 0); el.css(ref.position, distance / 2); } // Shift
+               var distance = (direction == 'vertical') ? wrapper.height() : wrapper.width();
+               if(mode == 'show') { wrapper.css(ref.size, 0); wrapper.css(ref.position, distance / 2); } // Shift
                
                // Animation
                var animation = {};
@@ -39,7 +39,7 @@ $.effects.clip = function(o) {
                animation[ref.position] = mode == 'show' ? 0 : distance / 2;
                        
                // Animate
-               el.animate(animation, { queue: false, duration: o.duration, easing: o.options.easing, complete: function() {
+               wrapper.animate(animation, { queue: false, duration: o.duration, easing: o.options.easing, complete: function() {
                        if(mode == 'hide') el.hide(); // Hide
                        $.effects.restore(el, props); $.effects.removeWrapper(el); // Restore
                        if(o.callback) o.callback.apply(this, arguments); // Callback