Skip to content

Commit

Permalink
effects.*: Explode effect was removing more elements than it should u…
Browse files Browse the repository at this point in the history
…pon completion. Fixed #6022 - multiple explosions cut short
  • Loading branch information
gnarf committed Mar 11, 2011
1 parent 19e7244 commit 4ade64b
Showing 1 changed file with 22 additions and 14 deletions.
36 changes: 22 additions & 14 deletions ui/jquery.effects.explode.js
Expand Up @@ -14,20 +14,22 @@

$.effects.explode = function( o ) {

return this.queue( function() {
return this.queue( function( next ) {

var rows = o.pieces ? Math.round(Math.sqrt(o.pieces)) : 3,
cells = rows,
el = $( this ).show().css( 'visibility', 'hidden' ),
mode = $.effects.setMode( el, o.mode || 'hide' ),
offset = el.offset(),
width = el.outerWidth( true ),
height = el.outerHeight( true );
height = el.outerHeight( true ),
peices = [];

//Substract the margins - not fixing the problem yet.
offset.top -= parseInt( el.css( "marginTop" ), 10 ) || 0;
offset.left -= parseInt( el.css( "marginLeft" ), 10 ) || 0;

// clone the element for each row and cell.
for( var i = 0; i < rows ; i++ ) { // =
for( var j = 0; j < cells ; j++ ) { // ||
el
Expand All @@ -54,23 +56,29 @@ $.effects.explode = function( o ) {
left: offset.left + j*(width/cells) + (o.mode == 'show' ? 0 : (j-Math.floor(cells/2))*(width/cells)),
top: offset.top + i*(height/rows) + (o.mode == 'show' ? 0 : (i-Math.floor(rows/2))*(height/rows)),
opacity: mode == 'show' ? 1 : 0
}, o.duration || 500);
}, o.duration || 500, childComplete );
}
}

// Set a timeout, to call the callback approx. when the other animations have finished
setTimeout(function() {
// children animate complete:
function childComplete() {
peices.push( this );
if ( peices.length == rows * cells ) {
animComplete();
}
}

function animComplete() {
el.css({ visibility: 'visible' });
mode != 'show' && el.hide();
$.isFunction( o.complete ) && o.complete.apply( el[ 0 ] );
el.dequeue();

// Note: This is removing all exploding peices from the dom, rather than the ones for this animation only... Ticket# 6022
$('div.ui-effects-explode').remove();
}, o.duration || 500);


$( peices ).remove();
if ( mode != 'show' ) {
el.hide();
}
if ( $.isFunction( o.complete ) ) {
o.complete.apply( el[ 0 ] );
}
next();
}
});

};
Expand Down

0 comments on commit 4ade64b

Please sign in to comment.