Skip to content

Conversation

prateekmedia
Copy link
Member

@prateekmedia prateekmedia commented Sep 26, 2025

Description

Make video editor experience better by using native api's. Resulting in more than 10x improvement in processing speed.

  • Use Media3 for android
  • Use iOS native apis

Tests

  • Test all functions for small videos on android
  • Test all functions for small videos on iOS
  • Test all functions for big videos on android
  • Test all functions for big videos on iOS

prateekmedia and others added 8 commits September 22, 2025 19:57
- Add useNativeVideoEditor flag that enables native video export only for internal users
- Update video_editor_page to check the flag and use FFmpeg for regular users
- Restore FFmpeg export functionality in ExportService.exportVideo()
- Native video editor remains available only for internal users and debug mode

Co-authored-by: Claude <[email protected]>
- Add Media3TransformerProcessor with proper Effects setup
- Use ScaleAndRotateTransformation for rotation operations
- Apply effects to EditedMediaItem using Effects class
- Fix import for androidx.media3.common.Effect
- Support trim, rotate, and crop operations in unified processor

Co-authored-by: Claude <[email protected]>
Modified video_editor package to support dimension overrides for proper
aspect ratio handling of rotated videos. Swaps width/height when rotation
metadata is 90/270 degrees and applies RotatedBox for correct display.

Co-authored-by: Claude <[email protected]>
- Use swapped dimensions for crop calculation when metadata rotation exists
- Fix crop coordinates to match dimensions used by crop grid viewer
- Change Media3 Presentation mode from SCALE_TO_FIT to SCALE_TO_FIT_WITH_CROP
- Pass metadata rotation from video editor to export service
- Add dimension override support to trim/crop/rotate pages

Co-Authored-By: Claude <[email protected]>
@prateekmedia prateekmedia changed the title [mob][photos] video editor revamped [mob][photos] native video editor Sep 26, 2025
@prateekmedia prateekmedia marked this pull request as draft September 26, 2025 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant