Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: Upgrade video_player_android to 2.7.3 #951

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rajveermalviya
Copy link
Collaborator

@rajveermalviya rajveermalviya commented Sep 18, 2024

Changelog:
https://pub.dev/packages/video_player_android/changelog

This update migrates the plugin to use newer API recommended
to support Impeller:
https://docs.flutter.dev/release/breaking-changes/android-surface-plugins

Without this migration, the videos do play but they look
pixelated, resulting in a bad UX.

before after
before after

Additionally, it updates the underlying Media3-ExoPlayer
library to version 1.4.1:
https://developer.android.com/jetpack/androidx/releases/media3#1.4.1

Changelog:
  https://pub.dev/packages/video_player_android/changelog

This update migrates the plugin to use newer API recommended
to support Impeller:
  https://docs.flutter.dev/release/breaking-changes/android-surface-plugins

Without this migration, the videos do play but they look
pixelated, resulting in a bad UX.

Additionally updates the underlying Media3-ExoPlayer
library to version 1.4.1:
  https://developer.android.com/jetpack/androidx/releases/media3#1.4.1
@rajveermalviya rajveermalviya added the maintainer review PR ready for review by Zulip maintainers label Sep 18, 2024
@chrisbobbe
Copy link
Collaborator

Sure, LGTM, thanks! Marking for Greg's review.

@chrisbobbe chrisbobbe added integration review Added by maintainers when PR may be ready for integration and removed maintainer review PR ready for review by Zulip maintainers labels Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration review Added by maintainers when PR may be ready for integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants