Twenty Twelve: Stop the navigation slide animation when clicking the button multiple times

[Jayjdk]

In Twenty Twelve, clicking the "Show navigation" button many times in a row causes a queue build up, where the navigation just slides up and down till it's done.

I've attached a patch that adds .stop() before .slideToggle()

"When .stop() is called on an element, the currently-running animation (if any) is immediately stopped" - ​http://api.jquery.com/stop/

[lancewillett]

Thanks for the report. Just for the sake of clarity, can you share how you debugged the queue build-up -- and steps to repeat it exactly?

[lancewillett]

In [21519]:

Twenty Twelve: stop nav slide animation when clicking the button multiple times in a row. Props Jayjdk, fixes #21531.

[Jayjdk]

I'm not sure that you're asking about. I noticed that the slide continued when I clicked the button multiple times. I remembered reading this article ​http://www.learningjquery.com/2009/01/quick-tip-prevent-animation-queue-buildup on how to prevent that.