fix: Keyboard event memory leak

jaywcjlove / hotkeys-js

➷ A robust Javascript library for capturing keyboard input. It has no dependencies.

https://jaywcjlove.github.io/hotkeys-js

MIT License

6.57k stars 405 forks source link

fix: Keyboard event memory leak #469

Closed qiongshusheng closed 6 months ago

qiongshusheng commented 6 months ago

问题：当绑定快捷键时会给元素添加keyup和keydown的事件，该事件一直没有被移除，即使元素上的快捷键都unbind了，也不会清除keyup和keydown事件，导致内存泄漏。解决方案：在unbind时判断unbind的元素上是否还有快捷键，如果没有快捷键就移除该元素上的keyup和keydown事件

解决 #90 提到的内存泄漏问题解决 #115 提到的在vue中使用引起的内存泄漏问题

jaywcjlove commented 6 months ago

@jaywcjlove thx!