Skip to content
Permalink
Browse files

Effects (fade): Delegate hiding and showing to core. Fixes #8267 - di…

…alog fadein broken since 1.9m4.
  • Loading branch information
scottgonzalez committed Jun 15, 2012
1 parent e2bbe04 commit d569b52838594c3e7e732d31dbf07dc2072edfdb
Showing with 4 additions and 11 deletions.
  1. +4 −11 ui/jquery.effects.fade.js
@@ -14,23 +14,16 @@

$.effects.effect.fade = function( o, done ) {
var el = $( this ),
mode = $.effects.setMode( el, o.mode || "toggle" ),
hide = mode === "hide";
mode = $.effects.setMode( el, o.mode || "toggle" );

el.show();
el.animate({
opacity: hide ? 0 : 1
opacity: mode
}, {
queue: false,
duration: o.duration,
easing: o.easing,
complete: function() {
if ( hide ) {
el.hide();
}
done();
}
complete: done
});
};

})(jQuery);
})( jQuery );

0 comments on commit d569b52

Please sign in to comment.
You can’t perform that action at this time.