From 1b26dea178560951106a5c59593f64c2766e84c7 Mon Sep 17 00:00:00 2001 From: lebaudantoine Date: Mon, 30 Jun 2025 21:00:38 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B(frontend)=20use=20feature=20detect?= =?UTF-8?q?ion=20for=20adaptiveStream=20and=20dynacast?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Replace hardcoded true values with supportsAdaptiveStream() and supportsDynacast() checks. LiveKit SDK supports broad browser range but requires specific APIs - modern features need explicit compatibility checks. Prevents enabling unsupported WebRTC features on incompatible browsers, which could led to a poor user experience. One alternative solution could be to install polyfills. --- .../src/features/rooms/components/Conference.tsx | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/src/frontend/src/features/rooms/components/Conference.tsx b/src/frontend/src/features/rooms/components/Conference.tsx index 891ce2e8..4e71df70 100644 --- a/src/frontend/src/features/rooms/components/Conference.tsx +++ b/src/frontend/src/features/rooms/components/Conference.tsx @@ -7,6 +7,8 @@ import { MediaDeviceFailure, Room, RoomOptions, + supportsAdaptiveStream, + supportsDynacast, } from 'livekit-client' import { keys } from '@/api/queryKeys' import { queryClient } from '@/api/queryClient' @@ -79,10 +81,13 @@ export const Conference = ({ retry: false, }) + const isAdaptiveStreamSupported = supportsAdaptiveStream() + const isDynacastSupported = supportsDynacast() + const roomOptions = useMemo((): RoomOptions => { return { - adaptiveStream: true, - dynacast: true, + adaptiveStream: isAdaptiveStreamSupported, + dynacast: isDynacastSupported, publishDefaults: { videoCodec: 'vp9', }, @@ -94,7 +99,12 @@ export const Conference = ({ }, } // do not rely on the userConfig object directly as its reference may change on every render - }, [userConfig.videoDeviceId, userConfig.audioDeviceId]) + }, [ + userConfig.videoDeviceId, + userConfig.audioDeviceId, + isAdaptiveStreamSupported, + isDynacastSupported, + ]) const room = useMemo(() => new Room(roomOptions), [roomOptions])