Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#4402 closed defect (fixed)

dojo._Animation delay param

Reported by: dante Owned by: Bryan Forbes
Priority: high Milestone: 1.0
Component: fx Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

the dojo._Animation pseudo-docs say delay: is a property you can mix in, though the normal practice is to set a delay in .play() ... setting a delay: attrib in any animation breaks the animation on mixin w/ animate property, though play(n) works fine.

seems like being able to mixin a delay specific to an _Animation (that won't have play(n) called on it) would be useful, eg:

dojo.fx.combine([

dojo.fadeIn({ delay:500, duration:500, node: node }), dojo.fadeOut({ duration:1000, node:otherNode })

]).play(25); or, after 25ms, play the _Animations: one 1000ms/0delay, one 500ms/500ms delay

it appears as though the fix to combine rests in making this work, too: possibly returning a new _Animation rather than trying to attach the the 'first' animation, clobbering it's unique onEnds, etc. (resuable stored animations, timing, etc).

Change History (2)

comment:1 Changed 12 years ago by dante

Resolution: fixed
Status: newclosed

delay was fixed by [10646] i suppose. combine is still broken, but that's not the scope of this bug.

comment:2 Changed 12 years ago by bill

Milestone: 1.11.0
Note: See TracTickets for help on using tickets.