refactor: align handling of default values

PRN-92
bitmovin · Dec 6, 2023 · 1fe872c · 1fe872c
1 parent 653fcd2
commit 1fe872c
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/components/PlayerView/index.tsx b/src/components/PlayerView/index.tsx
@@ -48,9 +48,9 @@ export function PlayerView({
   config,
   fullscreenHandler,
   customMessageHandler,
-  isFullscreenRequested = false,
+  isFullscreenRequested,
   scalingMode,
-  isPictureInPictureRequested = false,
+  isPictureInPictureRequested,
   ...props
 }: PlayerViewProps) {
   // Workaround React Native UIManager commands not sent until UI refresh
@@ -125,7 +125,7 @@ export function PlayerView({
 
   useEffect(() => {
     const node = findNodeHandle(nativeView.current);
-    if (node) {
+    if (node && isFullscreenRequested) {
       dispatch('setFullscreen', node, isFullscreenRequested);
     }
   }, [isFullscreenRequested, nativeView]);
@@ -139,7 +139,7 @@ export function PlayerView({
 
   useEffect(() => {
     const node = findNodeHandle(nativeView.current);
-    if (node) {
+    if (node && isPictureInPictureRequested) {
       dispatch('setPictureInPicture', node, isPictureInPictureRequested);
     }
   }, [isPictureInPictureRequested, nativeView]);