[Web] Fix `KeyboardEventManager` listeners unmount

software-mansion / react-native-gesture-handler

Declarative API exposing platform native touch and gesture system to React Native.

https://docs.swmansion.com/react-native-gesture-handler/

MIT License

6.13k stars 982 forks source link

[Web] Fix `KeyboardEventManager` listeners unmount #3126

Closed m-bert closed 1 month ago

m-bert commented 1 month ago

Description

For some reason KeyboardEventManager uses addEventListener inside unregisterListeners method. This PR changes it to removeEventListener, so that listeners are correctly removed from elements.

Test plan

I've tested that on my branch with removing findNodeHandle, so... you can trust me since PointerEventManager uses the same logic 😄