@applicaster/zapp-react-native-ui-components 14.0.0-alpha.2308642114 → 14.0.0-alpha.2332850672

package/Components/AudioPlayer/AudioPlayer.tsx

@@ -1,10 +1,16 @@
 import React, { useCallback, useMemo } from "react";
 
-import { platformSelect } from "@applicaster/zapp-react-native-utils/reactUtils";
+import {
+  platformSelect,
+  isTV,
+} from "@applicaster/zapp-react-native-utils/reactUtils";
 
 import { imageSrcFromMediaItem } from "@applicaster/zapp-react-native-utils/configurationUtils";
+import { getBackgroundImage } from "@applicaster/zapp-react-native-utils/audioPlayerUtils";
+
+import { AudioPlayerMobileLayout } from "./AudioPlayerMobileLayout";
+import { AudioPlayerTVLayout } from "./AudioPlayerTVLayout";
 
-import { AudioPlayerLayout } from "./AudioPlayerLayout";
 import { Channel } from "./Channel";
 import { Title } from "./Title";
 import { Summary } from "./Summary";
@@ -52,9 +58,20 @@ type Props = {
 };
 
 export function AudioPlayer(props: Props) {
-  const { audio_item, plugin_configuration, style } = props;
+  const { audio_item, plugin_configuration, style = {} } = props;
   const { extensions, title, summary } = audio_item;
 
+  const mobileConfig = useMemo(() => {
+    const backgroundImage = getBackgroundImage({
+      entry: audio_item,
+      plugin_configuration,
+    });
+
+    return {
+      backgroundImage,
+    };
+  }, [audio_item, plugin_configuration]);
+
   const getProp = useCallback(
     getPropertyFromEntryOrConfig({
       entry: audio_item,
@@ -63,12 +80,13 @@ export function AudioPlayer(props: Props) {
     [audio_item, plugin_configuration]
   );
 
-  const config = useMemo(() => {
+  const tvConfig = useMemo(() => {
     // Checking if we are recieving items from the DSP
     const titleColor = getProp("audio_player_title_color");
     const summaryColor = getProp("audio_player_summary_color");
     const backgroundColor = getProp("audio_player_background_color");
     const backgroundImage = getProp("audio_player_background_image");
+    const backgroundImageKey = getProp("audio_player_background_image_key");
     const artworkAspectRatio = getProp("audio_player_artwork_aspect_ratio");
     const channelIcon = getProp("audio_player_channel_icon");
     const rtlFlag = getProp("audio_player_rtl");
@@ -145,6 +163,7 @@ export function AudioPlayer(props: Props) {
       summaryColor,
       backgroundColor,
       backgroundImage,
+      backgroundImageKey,
       isRTL,
       titleFontFamily,
       titleFontSize,
@@ -160,15 +179,31 @@ export function AudioPlayer(props: Props) {
   }, [getProp]);
 
   const artwork = imageSrcFromMediaItem(audio_item, [
-    config?.artworkAspectRatio,
+    tvConfig?.artworkAspectRatio,
   ]);
 
+  console.log("debug_2", "AudioPlayer", {
+    tvConfig,
+    mobileConfig,
+    audio_item,
+    plugin_configuration,
+  });
+
+  if (isTV()) {
+    return (
+      <AudioPlayerTVLayout artwork={artwork} config={tvConfig} style={style}>
+        <Channel srcImage={tvConfig?.channelIcon} config={tvConfig} />
+        <Title title={title} config={tvConfig} />
+        <Summary summary={summary} config={tvConfig} />
+        <Runtime {...extensions} config={tvConfig} />
+      </AudioPlayerTVLayout>
+    );
+  }
+
   return (
-    <AudioPlayerLayout artwork={artwork} config={config} style={style || {}}>
-      <Channel srcImage={config?.channelIcon} config={config} />
-      <Title title={title} config={config} />
-      <Summary summary={summary} config={config} />
-      <Runtime {...extensions} config={config} />
-    </AudioPlayerLayout>
+    <AudioPlayerMobileLayout
+      backgroundImage={mobileConfig.backgroundImage}
+      style={style}
+    />
   );
 }

package/Components/AudioPlayer/AudioPlayerMobileLayout.tsx

@@ -0,0 +1,61 @@
+import React, { useRef } from "react";
+import {
+  View,
+  ImageBackground,
+  Animated,
+  ViewStyle,
+  StyleSheet,
+} from "react-native";
+import { platformSelect } from "@applicaster/zapp-react-native-utils/reactUtils";
+
+const styles = StyleSheet.create({
+  flex: {
+    flex: 1,
+  },
+});
+
+type Props = {
+  backgroundImage: string;
+  style: ViewStyle;
+};
+
+export function AudioPlayerMobileLayout({ backgroundImage, style }: Props) {
+  const fadeAnimation = useRef(new Animated.Value(0)).current;
+
+  const mainContainerStyles = platformSelect({
+    native: {
+      backgroundColor: "transparent",
+      overflow: "hidden",
+      ...style,
+    },
+  });
+
+  React.useEffect(() => {
+    Animated.timing(fadeAnimation, {
+      toValue: 1,
+      duration: 3000,
+      useNativeDriver: true,
+    }).start();
+  }, []);
+
+  return (
+    <View style={mainContainerStyles} pointerEvents="none">
+      <Animated.View
+        style={[
+          mainContainerStyles,
+          {
+            opacity: fadeAnimation,
+          },
+        ]}
+      >
+        <ImageBackground
+          source={{ uri: backgroundImage }}
+          style={styles.flex}
+          resizeMode="cover"
+        >
+          <View style={mainContainerStyles} />
+        </ImageBackground>
+      </Animated.View>
+    </View>
+  );
+}

package/Components/AudioPlayer/{AudioPlayerLayout.tsx → AudioPlayerTVLayout.tsx}

@@ -1,5 +1,5 @@
-import React, { useRef } from "react";
-import { View, ImageBackground, Animated, ViewStyle } from "react-native";
+import * as React from "react";
+import { View, ImageBackground, ViewStyle } from "react-native";
 
 import { platformSelect } from "@applicaster/zapp-react-native-utils/reactUtils";
 
@@ -20,17 +20,12 @@ type Props = {
   style: ViewStyle;
 };
 
-export function AudioPlayerLayout({ artwork, config, children, style }: Props) {
-  const fadeAnimation = useRef(new Animated.Value(0)).current;
-
-  const fadeAudioPlayerIn = () => {
-    Animated.timing(fadeAnimation, {
-      toValue: 1,
-      duration: 3000,
-      useNativeDriver: true,
-    }).start();
-  };
-
+export function AudioPlayerTVLayout({
+  artwork,
+  config,
+  children,
+  style,
+}: Props) {
   const { isRTL, backgroundColor, backgroundImage } = config;
 
   const backgroundImageSource = { uri: backgroundImage };
@@ -119,9 +114,6 @@ export function AudioPlayerLayout({ artwork, config, children, style }: Props) {
       alignItems: "center",
       justifyContent: "center",
     },
-    native: {
-      flex: 1,
-    },
   });
 
   const textContainerStyles = platformSelect({
@@ -145,58 +137,25 @@ export function AudioPlayerLayout({ artwork, config, children, style }: Props) {
     },
   });
 
-  const audioPlayerLayoutTV = backgroundImageSource?.uri ? (
-    <ImageBackground
-      source={backgroundImageSource}
-      style={backgroundImgStyles}
-      resizeMode="cover"
-    >
-      <View style={mainContainerStyles}>
-        {!!artwork && <Artwork srcImage={artwork} config={config} />}
-        <View style={textContainerStyles}>{children}</View>
-      </View>
-    </ImageBackground>
-  ) : (
+  if (backgroundImageSource?.uri) {
+    return (
+      <ImageBackground
+        source={backgroundImageSource}
+        style={backgroundImgStyles}
+        resizeMode="cover"
+      >
+        <View style={mainContainerStyles}>
+          {!!artwork && <Artwork srcImage={artwork} config={config} />}
+          <View style={textContainerStyles}>{children}</View>
+        </View>
+      </ImageBackground>
+    );
+  }
+
+  return (
     <View style={mainContainerStyles}>
       {!!artwork && <Artwork srcImage={artwork} config={config} />}
       <View style={textContainerStyles}>{children}</View>
     </View>
   );
-
-  const audioPlayerLayoutMobile = () => {
-    fadeAudioPlayerIn();
-
-    return (
-      <View style={mainContainerStyles} pointerEvents="none">
-        <Animated.View
-          style={[
-            mainContainerStyles,
-            {
-              opacity: fadeAnimation,
-            },
-          ]}
-        >
-          <ImageBackground
-            source={backgroundImageSource}
-            style={backgroundImgStyles}
-            resizeMode="cover"
-          >
-            <View style={mainContainerStyles} />
-          </ImageBackground>
-        </Animated.View>
-      </View>
-    );
-  };
-
-  const audioPlayerLayout = platformSelect({
-    tvos: audioPlayerLayoutTV,
-    android_tv: audioPlayerLayoutTV,
-    web: audioPlayerLayoutTV,
-    samsung_tv: audioPlayerLayoutTV,
-    lg_tv: audioPlayerLayoutTV,
-    ios: audioPlayerLayoutMobile(),
-    android: audioPlayerLayoutMobile(),
-  });
-
-  return audioPlayerLayout;
 }

package/Components/AudioPlayer/__tests__/__snapshots__/{audioPlayerLayout.test.js.snap → audioPlayerMobileLayout.test.js.snap}

@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`<AudioPlayerLayout /> renders correctly 1`] = `
+exports[`<AudioPlayerMobileLayout /> renders correctly 1`] = `
 <View
   pointerEvents="none"
   style={
@@ -32,7 +32,7 @@ exports[`<AudioPlayerLayout /> renders correctly 1`] = `
         resizeMode="cover"
         source={
           {
-            "uri": "https://example.com",
+            "uri": undefined,
           }
         }
         style={

package/Components/AudioPlayer/__tests__/audioPlayerMobileLayout.test.js

@@ -0,0 +1,18 @@
+import React from "react";
+import { render } from "@testing-library/react-native";
+
+import { AudioPlayerMobileLayout } from "../AudioPlayerMobileLayout";
+
+const audioPlayerLayoutProps = {
+  backgroundColor: "black",
+};
+
+describe("<AudioPlayerMobileLayout />", () => {
+  it("renders correctly", () => {
+    const { toJSON } = render(
+      <AudioPlayerMobileLayout {...audioPlayerLayoutProps} />
+    );
+
+    expect(toJSON()).toMatchSnapshot();
+  });
+});

package/Components/GeneralContentScreen/GeneralContentScreen.tsx

@@ -1,5 +1,4 @@
 import React, { useEffect } from "react";
-import { View, StyleSheet } from "react-native";
 import { ComponentsMap } from "../River/ComponentsMap";
 import { CellTapContext } from "@applicaster/zapp-react-native-ui-components/Contexts/CellTapContext";
 import { useActions } from "@applicaster/zapp-react-native-utils/reactHooks/actions";
@@ -13,23 +12,12 @@ import { createLogger } from "@applicaster/zapp-react-native-utils/logger";
 import { isNilOrEmpty } from "@applicaster/zapp-react-native-utils/reactUtils/helpers";
 import { ScreenTrackedViewPositionsContext } from "@applicaster/zapp-react-native-ui-components/Contexts/ScreenTrackedViewPositionsContext";
 import { useEventAlerts } from "./utils/useEventAlerts";
-import { useScreenBackgroundColor } from "@applicaster/zapp-react-native-utils/reactHooks/screen";
 
 const { log_info } = createLogger({
   category: "ScreenContainer",
   subsystem: "General",
 });
 
-const styles = StyleSheet.create({
-  background: {
-    position: "absolute",
-    left: 0,
-    bottom: 0,
-    right: 0,
-    top: 0,
-  },
-});
-
 export const GeneralContentScreen = ({
   feed,
   screenId,
@@ -42,7 +30,6 @@ export const GeneralContentScreen = ({
   isScreenWrappedInContainer,
   componentsMapExtraProps = {},
   focused,
-  extraOffset,
   parentFocus,
   containerHeight,
   preferredFocus = false,
@@ -50,12 +37,6 @@ export const GeneralContentScreen = ({
   const screenData = useScreenData(screenId);
 
   const uiComponents = useCurationAPI(screenData?.ui_components);
-  const backgroundColor = useScreenBackgroundColor(screenId);
-
-  const shouldShowBackground = React.useMemo(
-    () => backgroundColor && backgroundColor !== "transparent",
-    [backgroundColor]
-  );
 
   const onCellTapAction = useActions(
     whenMatchingType(String, cellTapAction) ||
@@ -127,30 +108,24 @@ export const GeneralContentScreen = ({
   if (!isReady || isNilOrEmpty(components || uiComponents)) return null;
 
   return (
-    <>
-      {shouldShowBackground ? (
-        <View style={[styles.background, { backgroundColor }]} />
-      ) : null}
-      <ScreenTrackedViewPositionsContext.Provider>
-        <CellTapContext.Provider value={contextValue}>
-          <ComponentsMap
-            feed={feed}
-            riverId={screenId}
-            groupId={groupId || `general-content-screen-${screenId}`}
-            riverComponents={components || uiComponents}
-            scrollViewExtraProps={scrollViewExtraProps}
-            getStaticComponentFeed={getStaticComponentFeed}
-            extraAnchorPointYOffset={extraAnchorPointYOffset}
-            isScreenWrappedInContainer={isScreenWrappedInContainer}
-            parentFocus={parentFocus}
-            focused={focused}
-            extraOffset={extraOffset}
-            containerHeight={containerHeight}
-            preferredFocus={preferredFocus}
-            {...componentsMapExtraProps}
-          />
-        </CellTapContext.Provider>
-      </ScreenTrackedViewPositionsContext.Provider>
-    </>
+    <ScreenTrackedViewPositionsContext.Provider>
+      <CellTapContext.Provider value={contextValue}>
+        <ComponentsMap
+          feed={feed}
+          riverId={screenId}
+          groupId={groupId || `general-content-screen-${screenId}`}
+          riverComponents={components || uiComponents}
+          scrollViewExtraProps={scrollViewExtraProps}
+          getStaticComponentFeed={getStaticComponentFeed}
+          extraAnchorPointYOffset={extraAnchorPointYOffset}
+          isScreenWrappedInContainer={isScreenWrappedInContainer}
+          parentFocus={parentFocus}
+          focused={focused}
+          containerHeight={containerHeight}
+          preferredFocus={preferredFocus}
+          {...componentsMapExtraProps}
+        />
+      </CellTapContext.Provider>
+    </ScreenTrackedViewPositionsContext.Provider>
   );
 };

package/Components/MasterCell/DefaultComponents/Text/index.tsx

@@ -52,6 +52,7 @@ const _Text = ({
         withScaledLineHeight(withFocusedStyles({ style, otherProps })),
         { height },
       ]}
+      allowFontScaling={false}
       {...withoutLabel(otherProps)}
     >
       {dateTransformEnabled

package/Components/PlayerContainer/PlayerContainer.tsx

@@ -135,6 +135,9 @@ const webStyles = {
   },
   inlineRiver: {
     height: INLINE_CONTAINER_CONTENT_HEIGHT,
+
+    borderWidth: 4,
+    borderColor: "yellow",
   },
 };
 
@@ -720,7 +723,7 @@ const PlayerContainerComponent = (props: Props) => {
                             isScreenWrappedInContainer={true}
                             containerHeight={styles.inlineRiver.height}
                             componentsMapExtraProps={{
-                              isNestedComponentsMap: R.T,
+                              isNestedComponentsMap: true,
                             }}
                           />
                         )}

package/Components/River/ComponentsMap/ComponentsMap.tsx

@@ -281,8 +281,8 @@ function ComponentsMapComponent(props: Props) {
             scrollIndicatorInsets={scrollIndicatorInsets}
             extraData={feed}
             stickyHeaderIndices={stickyHeaderIndices}
-            onLayout={handleOnLayout}
             removeClippedSubviews={isAndroid}
+            onLayout={handleOnLayout}
             initialNumToRender={3}
             maxToRenderPerBatch={10}
             windowSize={12}
@@ -303,10 +303,6 @@ function ComponentsMapComponent(props: Props) {
             onMomentumScrollEnd={_onMomentumScrollEnd}
             onScrollEndDrag={_onScrollEndDrag}
             scrollEventThrottle={16}
-            maintainVisibleContentPosition={{
-              minIndexForVisible: 0,
-              autoscrollToTopThreshold: 10,
-            }}
             {...scrollViewExtraProps}
           />
         </ViewportTracker>

package/Components/River/RiverItem.tsx

@@ -39,6 +39,10 @@ function getFeedUrl(feed: ZappFeed, index: number) {
   }
 }
 
+const isNextIndex = (index, readyIndex) => {
+  return readyIndex + 1 >= index;
+};
+
 /**
  * useLoadingState for RiverItemComponent
  * takes currentIndex and loadingState as arguments
@@ -48,24 +52,20 @@ const useLoadingState = (
   loadingState: RiverItemType["loadingState"]
 ) => {
   const [readyToBeDisplayed, setReadyToBeDisplayed] = React.useState(
-    loadingState.getValue().index + 1 >= currentIndex
+    isNextIndex(currentIndex, loadingState.getValue().index)
   );
 
   useEffect(() => {
     const subscription = loadingState.subscribe(({ index }) => {
-      if (index + 1 >= currentIndex) {
+      if (isNextIndex(currentIndex, index)) {
         setReadyToBeDisplayed(true);
       }
     });
 
-    if (loadingState.getValue().index + 1 >= currentIndex) {
-      setReadyToBeDisplayed(true);
-    }
-
     return () => {
       subscription.unsubscribe();
     };
-  }, [loadingState, currentIndex]);
+  }, [currentIndex]);
 
   return readyToBeDisplayed;
 };
@@ -151,7 +151,7 @@ function RiverItemComponent(props: RiverItemType) {
       component={item}
       componentIndex={index}
       onLoadFailed={onLoadFailed}
-      onLoadFinished={() => onLoadFinished(index)} // Keeping it here to don't break the plugins.
+      onLoadFinished={() => onLoadFinished(index)}
       groupId={groupId}
       feedUrl={feedUrl}
       isLast={isLast}

package/Components/River/TV/River.tsx

@@ -26,7 +26,6 @@ type Props = {
   componentsMapExtraProps?: any;
   isInsideContainer?: boolean;
   extraAnchorPointYOffset: number;
-  extraOffset: number;
   river?: ZappRiver | ZappEntry;
 };
 
@@ -39,7 +38,6 @@ export const River = (props: Props) => {
     componentsMapExtraProps,
     isInsideContainer,
     extraAnchorPointYOffset,
-    extraOffset,
   } = props;
 
   const { title: screenTitle, summary: screenSummary } = useNavbarState();
@@ -120,7 +118,6 @@ export const River = (props: Props) => {
     <GeneralContentScreen
       feed={feedData}
       screenId={screenId}
-      extraOffset={extraOffset}
       isScreenWrappedInContainer={isInsideContainer}
       extraAnchorPointYOffset={extraAnchorPointYOffset}
       componentsMapExtraProps={componentsMapExtraProps}

package/Components/River/__tests__/__snapshots__/componentsMap.test.js.snap

@@ -135,12 +135,6 @@ exports[`componentsMap renders renders components map correctly 1`] = `
       getItemCount={[Function]}
       initialNumToRender={3}
       keyExtractor={[Function]}
-      maintainVisibleContentPosition={
-        {
-          "autoscrollToTopThreshold": 10,
-          "minIndexForVisible": 0,
-        }
-      }
       maxToRenderPerBatch={10}
       onContentSizeChange={[Function]}
       onLayout={[Function]}

package/Components/TopMarginApplicator/TopMarginApplicator.tsx

@@ -1,17 +1,17 @@
 import React from "react";
-import { View, ViewProps, ViewStyle } from "react-native";
+import { View, ViewStyle } from "react-native";
 import { useTheme } from "@applicaster/zapp-react-native-utils/theme";
 import { useCurrentScreenData } from "@applicaster/zapp-react-native-utils/reactHooks";
 import { isFirstComponentScreenPicker } from "@applicaster/zapp-react-native-utils/componentsUtils";
+import { toNumberWithDefault } from "@applicaster/zapp-react-native-utils/numberUtils";
 
 interface IProps {
   targetScreenId?: string;
   children?: React.ReactNode;
   style?: ViewStyle;
-  extraOffset?: number;
+  extraVerticalOffset: Option<number>;
 }
 
-type CombinedProps = IProps & ViewProps;
 /**
  * The MarginTop is essentially a feature used for managing the visibility of components on your screen.
  * A more accurate term for this might be something like a 'component visibility threshold' or 'cut-off point'.
@@ -47,7 +47,7 @@ export const useMarginTop = (targetScreenId: string): number => {
 
   // Empty string means that value is blank in the CMS. Fallback to theme
   if (String(screenData?.styles?.screen_margin_top) === "") {
-    return Number(theme.screen_margin_top);
+    return toNumberWithDefault(0, theme.screen_margin_top);
   }
 
   /**
@@ -58,28 +58,29 @@ export const useMarginTop = (targetScreenId: string): number => {
    */
   if (screenData?.styles?.screen_margin_top === undefined) {
     if (isGeneralContentScreen || supportsUiComponents) {
-      return Number(theme.screen_margin_top);
+      return toNumberWithDefault(0, theme.screen_margin_top);
     }
 
     return 0;
   }
 
-  return Number(screenData?.styles?.screen_margin_top);
+  return toNumberWithDefault(0, screenData?.styles?.screen_margin_top);
 };
 
-export const TopMarginApplicator: React.FC<CombinedProps> = (
-  props: CombinedProps
-) => {
-  const extraOffset = props?.extraOffset ?? 0;
+export const TopMarginApplicator: React.FC<IProps> = ({
+  targetScreenId,
+  style,
+  children,
+  extraVerticalOffset,
+}: IProps) => {
+  const extraOffset = toNumberWithDefault(0, extraVerticalOffset);
 
   // HACK: Remove extraOffset when focusIssue with absolute elements is fixed on tvos
-  const marginTop = useMarginTop(props.targetScreenId) + extraOffset;
-  const style = { ...((props.style as {}) || {}), marginTop };
+  const marginTop = useMarginTop(targetScreenId);
 
-  // Then, spread the rest of the props on your returned JSX.
   return (
-    <View {...props} style={style}>
-      {props.children}
+    <View style={[style, { marginTop: marginTop + extraOffset }]}>
+      {children}
     </View>
   );
 };

package/Components/VideoModal/__tests__/__snapshots__/PlayerWrapper.test.tsx.snap

@@ -71,6 +71,36 @@ exports[`PlayerWrapper renders inline 1`] = `
         }
         testID="test-player-container"
       />
+      <View
+        animationType="componentFade"
+        style={
+          {
+            "flex": 1,
+          }
+        }
+      >
+        <View
+          configuration={
+            {
+              "tablet_landscape_player_container_background_color": "red",
+              "tablet_landscape_sidebar_width": "35%",
+            }
+          }
+          entry={
+            {
+              "id": "test",
+            }
+          }
+          isTablet={false}
+          isTabletLandscape={false}
+          style={
+            {
+              "flex": 1,
+              "paddingTop": 20,
+            }
+          }
+        />
+      </View>
     </View>
   </RNCSafeAreaView>
 </RNCSafeAreaProvider>
@@ -239,6 +269,36 @@ exports[`PlayerWrapper renders inline on tablet in landscape orientation 1`] = `
           }
         />
       </View>
+      <View
+        animationType="componentFade"
+        style={
+          {
+            "flex": 1,
+          }
+        }
+      >
+        <View
+          configuration={
+            {
+              "tablet_landscape_player_container_background_color": "red",
+              "tablet_landscape_sidebar_width": "35%",
+            }
+          }
+          entry={
+            {
+              "id": "test",
+            }
+          }
+          isTablet={true}
+          isTabletLandscape={true}
+          style={
+            {
+              "flex": 1,
+              "paddingTop": 20,
+            }
+          }
+        />
+      </View>
     </View>
   </RNCSafeAreaView>
 </RNCSafeAreaProvider>

package/Components/VideoModal/hooks/__tests__/useDelayedPlayerDetails.test.ts

@@ -1,19 +1,9 @@
 import { renderHook } from "@testing-library/react-hooks";
 import { useDelayedPlayerDetails } from "../useDelayedPlayerDetails";
-import { withTimeout$ } from "@applicaster/zapp-react-native-utils/idleUtils";
-import { showDetails } from "../utils";
 import { useIsTablet } from "@applicaster/zapp-react-native-utils/reactHooks";
 
-jest.mock("@applicaster/zapp-react-native-utils/idleUtils", () => ({
-  withTimeout$: jest.fn(),
-}));
-
-jest.mock("../utils", () => ({
-  showDetails: jest.fn(),
-}));
-
 jest.mock("@applicaster/zapp-react-native-utils/reactHooks", () => ({
-  useIsTablet: jest.fn(),
+  useIsTablet: jest.fn().mockReturnValue(false),
 }));
 
 describe("useDelayedPlayerDetails", () => {
@@ -21,69 +11,41 @@ describe("useDelayedPlayerDetails", () => {
     jest.clearAllMocks();
   });
 
-  it("should return false initially", () => {
-    (withTimeout$ as jest.Mock).mockReturnValue({
-      subscribe: jest.fn().mockReturnValue({ unsubscribe: jest.fn() }),
-    });
-
+  it("should return true initially", () => {
     const { result } = renderHook(() =>
       useDelayedPlayerDetails({ isInline: true, isDocked: false, isPip: false })
     );
 
-    expect(result.current).toBe(false);
+    expect(result.current).toBe(true);
   });
 
-  it("should return true if showDetails returns true", () => {
-    (withTimeout$ as jest.Mock).mockReturnValue({
-      subscribe: (callback) => {
-        callback.next();
-
-        return { unsubscribe: jest.fn() };
-      },
-    });
-
-    (showDetails as jest.Mock).mockReturnValue(true);
-    (useIsTablet as jest.Mock).mockReturnValue(false);
-
+  it("should return false when isPip is true", () => {
     const { result } = renderHook(() =>
-      useDelayedPlayerDetails({ isInline: true, isDocked: false, isPip: false })
+      useDelayedPlayerDetails({ isInline: true, isDocked: true, isPip: true })
     );
 
-    expect(result.current).toBe(true);
+    expect(result.current).toBe(false);
   });
 
-  it("should return false if showDetails returns false", () => {
-    (withTimeout$ as jest.Mock).mockReturnValue({
-      subscribe: (callback) => {
-        callback.next();
-
-        return { unsubscribe: jest.fn() };
-      },
-    });
-
-    (showDetails as jest.Mock).mockReturnValue(false);
-    (useIsTablet as jest.Mock).mockReturnValue(false);
-
+  it("should return false when isDocked is true", () => {
     const { result } = renderHook(() =>
-      useDelayedPlayerDetails({ isInline: true, isDocked: false, isPip: false })
+      useDelayedPlayerDetails({ isInline: true, isDocked: true, isPip: false })
     );
 
     expect(result.current).toBe(false);
   });
 
-  it("should unsubscribe on unmount", () => {
-    const unsubscribeMock = jest.fn();
+  it("should return true for tablet regardless of other flags", () => {
+    (useIsTablet as jest.Mock).mockReturnValue(true);
 
-    (withTimeout$ as jest.Mock).mockReturnValue({
-      subscribe: jest.fn().mockReturnValue({ unsubscribe: unsubscribeMock }),
-    });
-
-    const { unmount } = renderHook(() =>
-      useDelayedPlayerDetails({ isInline: true, isDocked: false, isPip: false })
+    const { result } = renderHook(() =>
+      useDelayedPlayerDetails({
+        isInline: false,
+        isDocked: false,
+        isPip: false,
+      })
     );
 
-    unmount();
-
-    expect(unsubscribeMock).toHaveBeenCalled();
+    expect(result.current).toBe(true);
   });
 });

package/Components/VideoModal/hooks/useDelayedPlayerDetails.ts

@@ -1,13 +1,23 @@
-import * as React from "react";
 import { useIsTablet } from "@applicaster/zapp-react-native-utils/reactHooks";
-import { withTimeout$ } from "@applicaster/zapp-react-native-utils/idleUtils";
 
 import { showDetails } from "./utils";
 
-const TIMEOUT = 100; // ms
-
 type Props = { isInline: boolean; isDocked: boolean; isPip: boolean };
 
+const showPlayerDetails = (
+  isInline: boolean,
+  isDocked: boolean,
+  isPip: boolean,
+  isTablet: boolean
+) => {
+  return showDetails({
+    isMobile: !isTablet,
+    isInline,
+    isDocked,
+    isPip,
+  });
+};
+
 /**
  * Custom hook to determine whether to show player details with a delay.
  *
@@ -22,28 +32,7 @@ export const useDelayedPlayerDetails = ({
   isDocked,
   isPip,
 }: Props): boolean => {
-  const [shouldShowDetails, setShouldShowDetails] = React.useState(false);
-
   const isTablet = useIsTablet();
 
-  React.useEffect(() => {
-    const subscription = withTimeout$(TIMEOUT).subscribe({
-      next: () => {
-        setShouldShowDetails(() => {
-          return showDetails({
-            isMobile: !isTablet,
-            isInline,
-            isDocked,
-            isPip,
-          });
-        });
-      },
-    });
-
-    return () => {
-      subscription.unsubscribe();
-    };
-  }, [isDocked, isTablet, isInline, isPip]);
-
-  return shouldShowDetails;
+  return showPlayerDetails(isInline, isDocked, isPip, isTablet);
 };

package/index.d.ts

@@ -24,7 +24,6 @@ type GeneralContentScreenProps = {
   } & Record<string, any>;
   focused?: boolean;
   parentFocus?: ParentFocus;
-  extraOffset?: number;
   containerHeight?: number;
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applicaster/zapp-react-native-ui-components",
-  "version": "14.0.0-alpha.2308642114",
+  "version": "14.0.0-alpha.2332850672",
   "description": "Applicaster Zapp React Native ui components for the Quick Brick App",
   "main": "index.js",
   "types": "index.d.ts",
@@ -31,10 +31,10 @@
     "redux-mock-store": "^1.5.3"
   },
   "dependencies": {
-    "@applicaster/applicaster-types": "14.0.0-alpha.2308642114",
-    "@applicaster/zapp-react-native-bridge": "14.0.0-alpha.2308642114",
-    "@applicaster/zapp-react-native-redux": "14.0.0-alpha.2308642114",
-    "@applicaster/zapp-react-native-utils": "14.0.0-alpha.2308642114",
+    "@applicaster/applicaster-types": "14.0.0-alpha.2332850672",
+    "@applicaster/zapp-react-native-bridge": "14.0.0-alpha.2332850672",
+    "@applicaster/zapp-react-native-redux": "14.0.0-alpha.2332850672",
+    "@applicaster/zapp-react-native-utils": "14.0.0-alpha.2332850672",
     "promise": "^8.3.0",
     "url": "^0.11.0",
     "uuid": "^3.3.2"

package/Components/AudioPlayer/__tests__/audioPlayerLayout.test.js

@@ -1,26 +0,0 @@
-import React from "react";
-import { render } from "@testing-library/react-native";
-
-import { AudioPlayerLayout } from "../AudioPlayerLayout";
-
-const audioPlayerLayoutProps = {
-  artwork: "string",
-  config: {
-    titleColor: "white",
-    summaryColor: "white",
-    backgroundColor: "black",
-    backgroundImage: "https://example.com",
-    isRTL: true,
-  },
-  children: [],
-};
-
-describe("<AudioPlayerLayout />", () => {
-  it("renders correctly", () => {
-    const { toJSON } = render(
-      <AudioPlayerLayout {...audioPlayerLayoutProps} />
-    );
-
-    expect(toJSON()).toMatchSnapshot();
-  });
-});