Fix Memory Leak: Remove all listeners when you destroy#93

Open

johnnyshields wants to merge 1 commit intoAutomattic:masterfrom

johnnyshields:patch-2

johnnyshields commented Apr 22, 2015

No description provided.


          Fix Memory Leak: Remove all listeners when you destroy

0a4f3f6

bcherny commented Oct 20, 2016

Jquery already does this when you call remove(). See https://github.com/jquery/jquery/blob/f18ca7bfe0f5e3184bf1ed55daf1668702c5577a/src/manipulation.js#L215 and https://github.com/jquery/jquery/blob/2d4f53416e5f74fa98e0c1d66b6f3c285a12f0ce/src/manipulation.js#L283

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet