-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Open
Description
Hi! π
Firstly, thanks for your work on this project! π
Today I used patch-package to patch [email protected] for the project I'm working on.
Here is the diff that solved my problem:
Original Issue
useEffect(() => {
let backHandler = () => {
callback();
return true;
};
if (enabled) {
BackHandler.addEventListener('hardwareBackPress', backHandler);
} else {
BackHandler.removeEventListener('hardwareBackPress', backHandler);
}
return () =>
BackHandler.removeEventListener('hardwareBackPress', backHandler);
}, [enabled, callback]);
Solution
useEffect(() => {
let backHandler = () => {
callback();
return true;
};
if (!enabled) return;
const backListener = BackHandler.addEventListener('hardwareBackPress', backHandler);
return () => backListener.remove();
}, [enabled, callback]);This issue body was partially generated by patch-package.
ChrisWiles, dotsenko-qarea, gomo, sebbalex, gitliyu and 1 more
Metadata
Metadata
Assignees
Labels
No labels