Skip to content

Commit e448290

Browse files
authored
Merge pull request #48 from pistou/patch-1
Improve unsubscription
2 parents 850d595 + c1ffdde commit e448290

File tree

1 file changed

+11
-22
lines changed

1 file changed

+11
-22
lines changed

src/useKeyboard.ts

Lines changed: 11 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -30,30 +30,19 @@ export default function useKeyboard() {
3030
}
3131

3232
useEffect(() => {
33-
const keyboardWillShowListener = Keyboard.addListener(
34-
'keyboardWillShow',
35-
handleKeyboardWillShow,
36-
)
37-
const keyboardDidShowListener = Keyboard.addListener(
38-
'keyboardDidShow',
39-
handleKeyboardDidShow,
40-
)
41-
const keyboardWillHideListener = Keyboard.addListener(
42-
'keyboardWillHide',
43-
handleKeyboardWillHide,
44-
)
45-
const keyboardDidHideListener = Keyboard.addListener(
46-
'keyboardDidHide',
47-
handleKeyboardDidHide,
48-
)
33+
Keyboard.addListener('keyboardWillShow', handleKeyboardWillShow);
34+
Keyboard.addListener('keyboardDidShow', handleKeyboardDidShow);
35+
Keyboard.addListener('keyboardWillHide', handleKeyboardWillHide);
36+
Keyboard.addListener('keyboardDidHide', handleKeyboardDidHide);
4937

5038
return () => {
51-
keyboardWillShowListener.remove()
52-
keyboardDidShowListener.remove()
53-
keyboardWillHideListener.remove()
54-
keyboardDidHideListener.remove()
55-
}
56-
}, [])
39+
Keyboard.removeListener('keyboardWillShow', handleKeyboardWillShow);
40+
Keyboard.removeListener('keyboardDidShow', handleKeyboardDidShow);
41+
Keyboard.removeListener('keyboardWillHide', handleKeyboardWillHide);
42+
Keyboard.removeListener('keyboardDidHide', handleKeyboardDidHide);
43+
};
44+
}, []);
45+
5746

5847
return {
5948
keyboardShown: shown,

0 commit comments

Comments
 (0)