onEnd() fires before animation finished

[bill]

When onEnd() is specified as a parameter to wipeOut(), wipeIn(), etc., the user defined callback executes before the animation is truly finished.

For example, this code from dijit/Tree.js:

wipeOut = fxUtils.wipeOut({
        node: this.containerNode,
        duration: manager.defaultDuration,
        onEnd: function(){
                def.resolve(true);
        }
});

It's due to this code in fx.js:

var fini = function(){
        s.overflow = o;
        s.height = "auto";
        s.display = "none";
};
connect.connect(anim, "onStop", fini);
connect.connect(anim, "onEnd", fini);

Since connect.connect() sets up after advice, fini() won't be called until after the user defined onEnd() method executes.

The workaround is for the user code to also use advice on the onEnd() method, rather than overriding it, but this is both inconvenient and non-intuitive.

Root cause of #16378.

[bill]

In [30123]:

Fix problem where collapseAll() and other asynchronous Tree methods resolve too early. Fixes #16378, refs #16431 !strict.

[dylan]

Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.