deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

jquery fx: closes #2542. show/hide(0, fn) wasn't calling the callback.

Browse source
This commit is contained in:
Ariel Flesler 2009-09-15 00:32:13 +00:00
parent 5e0da8c036
commit 321267dcec
2 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/fx.js
View file

@ -19,7 +19,7 @@ function genFx( type, num ){
jQuery.fn.extend({ jQuery.fn.extend({
show: function(speed,callback){ show: function(speed,callback){
if ( speed ) { if ( speed !== undefined ) {
return this.animate( genFx("show", 3), speed, callback); return this.animate( genFx("show", 3), speed, callback);
} else { } else {
for ( var i = 0, l = this.length; i < l; i++ ){ for ( var i = 0, l = this.length; i < l; i++ ){
@ -59,7 +59,7 @@ jQuery.fn.extend({
}, },
hide: function(speed,callback){ hide: function(speed,callback){
if ( speed ) { if ( speed !== undefined ) {
return this.animate( genFx("hide", 3), speed, callback); return this.animate( genFx("hide", 3), speed, callback);
} else { } else {
for ( var i = 0, l = this.length; i < l; i++ ){ for ( var i = 0, l = this.length; i < l; i++ ){

10
test/unit/fx.js
View file

@ -68,7 +68,7 @@ test("animate option (queue === false)", function () {
*/ */
test("animate duration 0", function() { test("animate duration 0", function() {
expect(5); expect(7);
stop(); stop();
@ -98,6 +98,14 @@ test("animate duration 0", function() {
equals( counter, 5, "One synchronic and one asynchronic" ); equals( counter, 5, "One synchronic and one asynchronic" );
start(); start();
}); });
var $elem = jQuery("<div />");
$elem.show(0, function(){
ok(true, "Show's callback with no duration");
});
$elem.hide(0, function(){
ok(true, "Show's callback with no duration");
});
}); });
test("animate hyphenated properties", function(){ test("animate hyphenated properties", function(){