From 53e96f9febad09a8a0a1bf274bcf74e7977c5826 Mon Sep 17 00:00:00 2001 From: Pavlo Buidenkov Date: Wed, 1 Jun 2022 18:10:48 +0200 Subject: [PATCH] fix connected devices drawer bug --- app/components/ConnectedDevicesListDrawer.tsx | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/app/components/ConnectedDevicesListDrawer.tsx b/app/components/ConnectedDevicesListDrawer.tsx index a248ccd..4c7337d 100644 --- a/app/components/ConnectedDevicesListDrawer.tsx +++ b/app/components/ConnectedDevicesListDrawer.tsx @@ -84,6 +84,12 @@ export default function ConnectedDevicesListDrawer( }); } setConnectedDevices(devicesWithSourceIds); + + const map = new Map(); + devicesWithSourceIds.forEach((el) => { + map.set(el.id, true); + }); + setDevicesDisplayed(map); }) // eslint-disable-next-line no-console .catch((e) => console.error(e)); @@ -101,14 +107,6 @@ export default function ConnectedDevicesListDrawer( }; }, []); - useEffect(() => { - const map = new Map(); - connectedDevices.forEach((el) => { - map.set(el.id, true); - }); - setDevicesDisplayed(map); - }, [setDevicesDisplayed, connectedDevices]); - const handleDisconnectOneDevice = useCallback( async (id: string) => { const device = connectedDevices.find((d: Device) => d.id === id); @@ -159,10 +157,10 @@ export default function ConnectedDevicesListDrawer( (id) => { setTimeout( async () => { - await handleDisconnectOneDevice(id); + handleDisconnectOneDevice(id); hideOneDeviceInDevicesDisplayed(id); }, - isProduction() ? 1000 : 0 + isProduction() ? ANIMATION_DURATION : 0 ); }, [handleDisconnectOneDevice, hideOneDeviceInDevicesDisplayed]