xmj/element-call
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Split out client sync listeber into separate useEffect

Browse source
This commit is contained in:
David Baker 2022-06-01 10:21:44 +01:00
parent 64e30c89e3
commit d9bd48b9a6
1 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/room/usePTT.ts
View file

@ -262,14 +262,10 @@ export const usePTT = (
window.addEventListener("keyup", onKeyUp); window.addEventListener("keyup", onKeyUp);
window.addEventListener("blur", onBlur); window.addEventListener("blur", onBlur);
client.on(ClientEvent.Sync, onClientSync);
return () => { return () => {
window.removeEventListener("keydown", onKeyDown); window.removeEventListener("keydown", onKeyDown);
window.removeEventListener("keyup", onKeyUp); window.removeEventListener("keyup", onKeyUp);
window.removeEventListener("blur", onBlur); window.removeEventListener("blur", onBlur);
client.removeListener(ClientEvent.Sync, onClientSync);
}; };
}, [ }, [
groupCall, groupCall,
@ -285,6 +281,14 @@ export const usePTT = (
onClientSync, onClientSync,
]); ]);
useEffect(() => {
client.on(ClientEvent.Sync, onClientSync);
return () => {
client.removeListener(ClientEvent.Sync, onClientSync);
};
}, [client, onClientSync]);
const setTalkOverEnabled = useCallback((talkOverEnabled) => { const setTalkOverEnabled = useCallback((talkOverEnabled) => {
setState((prevState) => ({ setState((prevState) => ({
...prevState, ...prevState,