Refactor video frame scaling logic in VideoEditor to simplify resizing process

This commit removes redundant checks for scaling dimensions and ensures that the video frame is resized correctly while maintaining the aspect ratio. The logic is streamlined to enhance clarity and efficiency in the resizing process, improving overall performance during video editing.

croppa/main.py

@@ -2880,38 +2880,53 @@ class VideoEditor:
         if display_frame is None:
             return
 
-        # Resize to fit window while maintaining aspect ratio
+        # Get actual window dimensions (important for fullscreen mode)
-        height, width = display_frame.shape[:2]
+        window_title = "Image Editor" if self.is_image_mode else "Video Editor"
-        available_height = self.window_height - (0 if self.is_image_mode else self.TIMELINE_HEIGHT)
+        try:
+            window_rect = cv2.getWindowImageRect(window_title)
+            if window_rect[2] > 0 and window_rect[3] > 0:  # width and height > 0
+                actual_width = window_rect[2]
+                actual_height = window_rect[3]
+            else:
+                # Fallback to stored dimensions
+                actual_width = self.window_width
+                actual_height = self.window_height
+        except:
+            # Fallback to stored dimensions
+            actual_width = self.window_width
+            actual_height = self.window_height
 
-        # Don't downscale - keep original video quality
+        # Calculate available space for video (accounting for timeline)
-        # If video is larger than window, we'll handle it by resizing the window
+        available_height = actual_height - (0 if self.is_image_mode else self.TIMELINE_HEIGHT)
-        scale = min(self.window_width / width, available_height / height)
+        
-        if scale < 1.0:
+        # Get original video dimensions to maintain aspect ratio
-            # Resize window to fit video instead of downscaling video
+        height, width = display_frame.shape[:2]
-            new_window_width = int(width * 1.1)  # Add 10% padding
+        
-            new_window_height = int(height * 1.1) + (0 if self.is_image_mode else self.TIMELINE_HEIGHT)
+        # Calculate scale to fit video in available space while maintaining aspect ratio
-            
+        scale_x = actual_width / width
-            # Update window size
+        scale_y = available_height / height
-            self.window_width = new_window_width
+        scale = min(scale_x, scale_y)  # Use the smaller scale to ensure video fits
-            self.window_height = new_window_height
+        
-            
+        # Calculate scaled dimensions
-            # Resize the OpenCV window
+        scaled_width = int(width * scale)
-            window_title = "Image Editor" if self.is_image_mode else "Video Editor"
+        scaled_height = int(height * scale)
-            cv2.resizeWindow(window_title, self.window_width, self.window_height)
+        
+        # Resize the frame to fit the window while maintaining aspect ratio
+        if scale != 1.0:
+            display_frame = cv2.resize(display_frame, (scaled_width, scaled_height), interpolation=cv2.INTER_LINEAR)
 
         # Create canvas with timeline space
-        canvas = np.zeros((self.window_height, self.window_width, 3), dtype=np.uint8)
+        canvas = np.zeros((actual_height, actual_width, 3), dtype=np.uint8)
 
         # Center the frame on canvas
         frame_height, frame_width = display_frame.shape[:2]
-        available_height = self.window_height - (0 if self.is_image_mode else self.TIMELINE_HEIGHT)
+        available_height = actual_height - (0 if self.is_image_mode else self.TIMELINE_HEIGHT)
         start_y = (available_height - frame_height) // 2
-        start_x = (self.window_width - frame_width) // 2
+        start_x = (actual_width - frame_width) // 2
 
         # Ensure frame fits within canvas bounds
         end_y = min(start_y + frame_height, available_height)
-        end_x = min(start_x + frame_width, self.window_width)
+        end_x = min(start_x + frame_width, actual_width)
         actual_frame_height = end_y - start_y
         actual_frame_width = end_x - start_x