mousetrap icon indicating copy to clipboard operation
mousetrap copied to clipboard

added `destroy`method to cleanup listener

Open olivierchatry opened this issue 5 years ago • 3 comments

Following https://github.com/ccampbell/mousetrap/issues/427

olivierchatry avatar Aug 16 '18 10:08 olivierchatry

agreed

lorrylockie avatar Feb 12 '19 03:02 lorrylockie

@ccampbell could you please have a look on this? Looks like missing ability to remove event listener can be a major drawback leading to a memory leak

besuhoff avatar May 13 '19 04:05 besuhoff

Echoing the comment above, not having this option leads to major memory leaks - @ccampbell apologies for the @, but can we look into merging this or a similar fix?

EBruchet avatar Dec 07 '23 11:12 EBruchet