fix: maintain aspect-ratio, re-enable visibility tracking (#1136)

GetStream · Oct 10, 2023 · 0c8319e · 0c8319e
1 parent fd96007
commit 0c8319e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/...ps/react/react-video-demo/src/components/ParticipantsSlider/ParticipantsSlider.module.css b/...ps/react/react-video-demo/src/components/ParticipantsSlider/ParticipantsSlider.module.css
@@ -20,7 +20,6 @@
 .participant.vertical {
   width: 100%;
   border-radius: 12px;
-  aspect-ratio: auto;
 }
 
 .participant.vertical video {

diff --git a/sample-apps/react/react-video-demo/src/components/ParticipantsSlider/ParticipantsSlider.tsx b/sample-apps/react/react-video-demo/src/components/ParticipantsSlider/ParticipantsSlider.tsx
@@ -72,10 +72,10 @@ export const ParticipantsSlider: FC<Props> = ({
   );
 
   useEffect(() => {
-    if (!scrollWrapper || !call || mode === 'vertical') return;
+    if (!scrollWrapper || !call) return;
     const cleanup = call.setViewport(scrollWrapper);
     return () => cleanup();
-  }, [scrollWrapper, call, mode]);
+  }, [scrollWrapper, call]);
 
   useEffect(() => {
     if (breakpoint === 'xs' || breakpoint === 'sm') {