.stop sometimes doesn't clear .delay

[cowboy]

If .stop is called after the setTimeout in .delay has executed, there's no way for that setTimeout to be stopped, so the callback executes after "time" milliseconds, no matter what.

[addyosmani]

Please submit a valid test case that reproduces this issue.

[cowboy]

Here you go, sorry I never added this into the ticket!

​http://jsfiddle.net/cowboy/xxvFa/

[SlexAxton]

Appreciate the test case, ben.

[john]

#6576 is a duplicate of this ticket.

[T.J. Crowder]

Just "me too"-ing. stop() should stop delays. It seems like it should be really easy to implement delay() as an animation of the given duration that doesn't actually change anything (if, unlike me, you already understand how jQuery's animation infrastructure works). That would make it play nicely with the various other animation functions (like stop()).

[john]

Nominating ticket for 1.7 discussion.

[gnarf]

You could just animate a non-property

$.fn.delay = function( duration ) {
    return this.animate({
       _: 0
    }, duration);

However, This brings up a great point.... stopHooks...

There are a lot of UI effects that CANT be .stop()'ed because they don't actually use the queue... It might be nice to create some sort of method that non-standard animates can provide a function to be called on stop...

Imagine:

$.fn.delay = function( duration ) {
    return this.queue( function( next, onstop ) {
       var timer = setTimeout( next, duration );
       onstop( function( gotoEnd ) {
         clearTimeout( timer );
         if ( gotoEnd ) {
            next();
          }
       });
    });
};

[Rick Waldron]

+1, Sounds like a bug, should be fixed

[jaubourg]

+1, Sounds like a nasty little bug

[ajpiano]

+1, Exterminate!

[Timmy Willison]

+1, could this be fixed by simply adding the setTimeout timer to jQuery.timers?

[danheberden]

+1

[dmethvin]

+1, Do not annoy the cowboy.

[john]

+1

[scottgonzalez]

+1, clearly just an implementation bug

[jzaefferer]

+1

[cowboy]

+1, heh

[Timmy Willison]

We are currently favoring a fix for this that would include an additional parameter for delay and considering delay has always behaved this way, we're going to push this to 1.7.

[Corey Frang]

Landing pull request 514. 1.7 - queue refactoring to handle delay stop - Fixes #6150.

More Details:

• ​https://github.com/jquery/jquery/pull/514
• ​http://bugs.jquery.com/ticket/6150 Changeset: a3b59d7f92c9e15af1888fc4e87639a290763a50