Skip to content

Commit 1973bf2

Browse files
authored
fix(NativeEngineHook): avoid deprecated warning (#472)
Fix useAppState hook causing "removeEventListener" deprecated warning on newer react native versions Tried to keep it backward compatible by checking if the remove method exists
1 parent c0a2841 commit 1973bf2

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

Modules/@babylonjs/react-native/NativeEngineHook.ts

+8-3
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,16 @@ function useAppState(): string {
3434
setAppState(appState);
3535
};
3636

37-
AppState.addEventListener("change", onAppStateChanged);
37+
const appStateListener = AppState.addEventListener("change", onAppStateChanged);
3838

3939
return () => {
40-
AppState.removeEventListener("change", onAppStateChanged);
41-
}
40+
if (!!appStateListener?.remove) {
41+
appStateListener.remove();
42+
}
43+
else {
44+
AppState.removeEventListener("change", onAppStateChanged);
45+
}
46+
};
4247
}, []);
4348

4449
return appState;

0 commit comments

Comments
 (0)