#6022 closed bug (fixed)
Effects:explode - multiple explosions cut short
Reported by: | stringfold | Owned by: | Corey Frang |
---|---|---|---|
Priority: | minor | Milestone: | 1.9.0 |
Component: | ui.effects.* (individual effect) | Version: | 1.8.4 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
If you trigger a second explode effect before the first is completed, all the animated image fragments for both animations are removed from the window when the first animation ends.
This is because the callback called when the first animation ends simply calls:
$('div.ui-effects-explode').remove();
which removes all the image fragments for every animation currently in progress.
This is a shame because everything else works for concurrent explosions works fine -- the animations and the callback function to the user code when the animation is supposed to end.
The fix would be to attach a unique identifier to each set of exploding image elements so the callback can selectively remove only the images for that specific animation.
Change History (8)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
Component: | ui.core → effects.* (individual effect) |
---|---|
Milestone: | TBD → 1.9 |
comment:4 Changed 13 years ago by
Owner: | set to Corey Frang |
---|---|
Status: | new → assigned |
comment:5 Changed 13 years ago by
Blocked by: | 7060 added |
---|
comment:6 Changed 13 years ago by
Blocked by: | 7060 removed |
---|
comment:7 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
effects.*: Explode effect was removing more elements than it should upon completion. Fixed #6022 - multiple explosions cut short
Changeset: 4ade64b69028178772d0b9c11fbf7165de160972
I should add that, if possible, you should be able to select the exploding pieces by the unique identifier -- e.g. for overlapping explosions, you would be able set the z-index to ensure the pieces were displayed in the correct order.
Perhaps an "id" option could be added to the effect API for the explode effect, which would be set to all the exploding pieces. Absent the setting of that option, the code would generate its own unique id to allow the correct removal of only the pieces associated with that explosion.