@umituz/react-native-video-editor 1.0.9 → 1.0.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-video-editor",
-  "version": "1.0.9",
+  "version": "1.0.11",
   "description": "Professional video editor with layer-based timeline, text/image/shape/audio/animation layers, and export functionality",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/player/infrastructure/services/player-control.service.ts

@@ -11,16 +11,16 @@ declare const __DEV__: boolean;
  * Safely play video with error handling
  */
 export const safePlay = (player: VideoPlayer | null): boolean => {
+  console.log("[safePlay] called, player:", !!player);
   if (!player) return false;
 
   try {
+    console.log("[safePlay] calling player.play()");
     player.play();
+    console.log("[safePlay] player.play() called successfully");
     return true;
   } catch (error) {
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      // eslint-disable-next-line no-console
-      console.log("[VideoPlayer] Play error ignored:", error);
-    }
+    console.log("[safePlay] Play error:", error);
     return false;
   }
 };

package/src/player/presentation/components/VideoPlayer.tsx

@@ -8,17 +8,31 @@ import {
   View,
   TouchableOpacity,
   StyleSheet,
-  useWindowDimensions,
+  type ViewStyle,
 } from "react-native";
 import { Image } from "expo-image";
 import { VideoView } from "expo-video";
-import { useAppDesignTokens, AtomicIcon } from "@umituz/react-native-design-system";
+import {
+  useAppDesignTokens,
+  AtomicIcon,
+  useResponsive,
+} from "@umituz/react-native-design-system";
 
 import type { VideoPlayerProps } from "../../types";
 import { useVideoPlayerControl } from "../hooks/useVideoPlayerControl";
 
+declare const __DEV__: boolean;
+
 const ASPECT_RATIO = 16 / 9;
 
+/** Extract numeric width from style prop */
+const getWidthFromStyle = (style: ViewStyle | undefined): number | null => {
+  if (!style) return null;
+  const w = style.width;
+  if (typeof w === "number") return w;
+  return null;
+};
+
 export const VideoPlayer: React.FC<VideoPlayerProps> = ({
   source,
   thumbnailUrl,
@@ -30,57 +44,65 @@ export const VideoPlayer: React.FC<VideoPlayerProps> = ({
   style,
 }) => {
   const tokens = useAppDesignTokens();
-  const { width } = useWindowDimensions();
+  const { width: screenWidth, horizontalPadding } = useResponsive();
   const [showVideo, setShowVideo] = useState(autoPlay);
 
   const { player, state, controls } = useVideoPlayerControl({
-    source: showVideo ? source : null,
+    source,
     loop,
     muted,
-    autoPlay: showVideo ? true : autoPlay,
+    autoPlay: false,
   });
 
-  // Auto-play when video becomes visible
   useEffect(() => {
     if (showVideo && state.isPlayerValid && player) {
-      console.log("[VideoPlayer] Playing video...");
+      if (typeof __DEV__ !== "undefined" && __DEV__) {
+        // eslint-disable-next-line no-console
+        console.log("[VideoPlayer] Starting playback...");
+      }
       controls.play();
     }
   }, [showVideo, state.isPlayerValid, player, controls]);
 
   const handlePlay = useCallback(() => {
-    console.log("[VideoPlayer] handlePlay called, source:", source);
+    if (typeof __DEV__ !== "undefined" && __DEV__) {
+      // eslint-disable-next-line no-console
+      console.log("[VideoPlayer] handlePlay called, source:", source);
+    }
     setShowVideo(true);
   }, [source]);
 
+  // Calculate absolute dimensions for VideoView (expo-video requires pixel values)
+  const videoWidth = getWidthFromStyle(style as ViewStyle) ?? (screenWidth - horizontalPadding * 2);
+  const videoHeight = videoWidth / ASPECT_RATIO;
+
   const containerStyle = useMemo(() => ({
-    width: "100%" as const,
-    aspectRatio: ASPECT_RATIO,
+    width: videoWidth,
+    height: videoHeight,
     backgroundColor: tokens.colors.surface,
     borderRadius: 16,
     overflow: "hidden" as const,
-  }), [tokens.colors.surface]);
+  }), [tokens.colors.surface, videoWidth, videoHeight]);
 
   const styles = useMemo(
     () =>
       StyleSheet.create({
         video: {
-          width: "100%",
-          height: "100%",
+          width: videoWidth,
+          height: videoHeight,
         },
         thumbnailContainer: {
-          width: "100%",
-          height: "100%",
+          flex: 1,
           justifyContent: "center",
           alignItems: "center",
         },
         thumbnail: {
-          width: "100%",
-          height: "100%",
+          width: videoWidth,
+          height: videoHeight,
         },
         placeholder: {
-          width: "100%",
-          height: "100%",
+          width: videoWidth,
+          height: videoHeight,
           backgroundColor: tokens.colors.surfaceSecondary,
         },
         playButtonContainer: {
@@ -98,20 +120,24 @@ export const VideoPlayer: React.FC<VideoPlayerProps> = ({
           paddingLeft: 4,
         },
       }),
-    [tokens]
+    [tokens, videoWidth, videoHeight]
   );
 
-  // Debug: Log player state
-  console.log("[VideoPlayer] state:", {
-    showVideo,
-    isPlayerValid: state.isPlayerValid,
-    hasPlayer: !!player,
-    source,
-  });
+  if (typeof __DEV__ !== "undefined" && __DEV__) {
+    // eslint-disable-next-line no-console
+    console.log("[VideoPlayer] state:", {
+      showVideo,
+      isPlayerValid: state.isPlayerValid,
+      hasPlayer: !!player,
+      source,
+    });
+  }
 
-  // Show video player when playing
   if (showVideo && state.isPlayerValid && player) {
-    console.log("[VideoPlayer] Rendering VideoView");
+    if (typeof __DEV__ !== "undefined" && __DEV__) {
+      // eslint-disable-next-line no-console
+      console.log("[VideoPlayer] Rendering VideoView");
+    }
     return (
       <View style={[containerStyle, style]}>
         <VideoView
@@ -124,7 +150,6 @@ export const VideoPlayer: React.FC<VideoPlayerProps> = ({
     );
   }
 
-  // Show thumbnail with play button
   return (
     <TouchableOpacity
       style={[containerStyle, style]}

package/src/player/presentation/hooks/useVideoPlayerControl.ts

@@ -31,7 +31,8 @@ export const useVideoPlayerControl = (
   const [isPlaying, setIsPlaying] = useState(false);
   const [isLoading, setIsLoading] = useState(true);
 
-  const player = useExpoVideoPlayer(source ?? "", (p) => {
+  const player = useExpoVideoPlayer(source || "", (p) => {
+    console.log("[useVideoPlayerControl] Player callback, source:", source, "player:", !!p);
     if (source && p) {
       configurePlayer(p, { loop, muted, autoPlay });
       setIsLoading(false);