See [[en:Wikipedia:Village pump (technical)#Can't go directly to feedback pages]].

The 'stall' method installs an event handler that calls a callback which is expected to return a Deferred object, queues up a "re-fire the event" function for when that Deferred object resolves, and then returns false for the original event.

But if the callback returns something other than a Deferred object (e.g. as AFTv5 does if $.aftTrack.clickTrackingOn is false), returns an already-resolved Deferred object, or throws an exception, stall will try to re-fire the event before the original event handler returns. This causes problems in IE at least for the 'click' event, as the re-fired click is ignored.

Also, it seems if the Deferred object returned by the callback is rejected, stall will never re-fire the event since it only attaches a "done" handler and not a "fail" handler. I don't know whether or not this is intended behavior.

Version: master
Severity: normal