diff options
author | Aaron Eisenberger <aaronchi@gmail.com> | 2008-06-16 13:18:48 +0000 |
---|---|---|
committer | Aaron Eisenberger <aaronchi@gmail.com> | 2008-06-16 13:18:48 +0000 |
commit | 8ec7ceea10d62abafbf79ea0f21ff31a2085aa44 (patch) | |
tree | b6658126bc1219cf1a0411a2f5fa930ff51a5d58 | |
parent | 391c6561efea4c96cbb95ed95c9200f4f3279b41 (diff) | |
download | jquery-ui-8ec7ceea10d62abafbf79ea0f21ff31a2085aa44.tar.gz jquery-ui-8ec7ceea10d62abafbf79ea0f21ff31a2085aa44.zip |
clip effect should be animating the wrapper
-rw-r--r-- | ui/effects.clip.js | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/ui/effects.clip.js b/ui/effects.clip.js index d1329f46e..6bfeddeb3 100644 --- a/ui/effects.clip.js +++ b/ui/effects.clip.js @@ -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 |