From 51926cad3d39e89127ed8a14190fbe8cb5859a12 Mon Sep 17 00:00:00 2001 From: Timo <16718859+toger5@users.noreply.github.com> Date: Tue, 30 Apr 2024 20:08:39 +0200 Subject: [PATCH] Revert "Fix mute flickering. (#2350)" (#2351) This reverts commit 2ab909fab16ad2019794317ee3b2dffde2e01717. --- src/room/VideoPreview.tsx | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/src/room/VideoPreview.tsx b/src/room/VideoPreview.tsx index 547db7f8..602ca5e5 100644 --- a/src/room/VideoPreview.tsx +++ b/src/room/VideoPreview.tsx @@ -68,8 +68,8 @@ export const VideoPreview: FC = ({ deviceId: devices.audioInput.selectedId, }; - const deviceConfig = useMemo(() => { - return { + const tracks = usePreviewTracks( + { // The only reason we request audio here is to get the audio permission // request over with at the same time. But changing the audio settings // shouldn't cause this hook to recreate the track, which is why we @@ -80,15 +80,13 @@ export const VideoPreview: FC = ({ video: muteStates.video.enabled && { deviceId: devices.videoInput.selectedId, }, - }; - }, [devices.videoInput.selectedId, muteStates.video.enabled]); - - const tracks = usePreviewTracks(deviceConfig, (error) => { - logger.error("Error while creating preview Tracks:", error); - muteStates.audio.setEnabled?.(false); - muteStates.video.setEnabled?.(false); - }); - + }, + (error) => { + logger.error("Error while creating preview Tracks:", error); + muteStates.audio.setEnabled?.(false); + muteStates.video.setEnabled?.(false); + }, + ); const videoTrack = useMemo( () => tracks?.find((t) => t.kind === Track.Kind.Video) as