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

Support different resolutions of video #492

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

merdweebner
Copy link

lib/src/models/video_player_controller_extension.dart

@Ahmadre Ahmadre linked an issue Jun 13, 2021 that may be closed by this pull request
@Ahmadre Ahmadre self-assigned this Jun 13, 2021
@@ -179,6 +181,21 @@ class _MaterialControlsState extends State<MaterialControls>
);
}

if (chewieController.resolutions != null &&
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace with if (chewieController.resolutions?.isNotEmpty ?? false) {.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx will do it

Future<void> _onResolutionTap() async {
_hideTimer?.cancel();

final choosenResolution = await showModalBottomSheet<String>(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename variable to chosenResolution.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx will do it

@Ahmadre Ahmadre changed the title add copyWith extension | add resolution dialog Support different resolutions of video Apr 16, 2022
@AhmadFalahian
Copy link

Can I use this branch without using master branch?

@anwar907
Copy link

anwar907 commented Mar 3, 2023

Hi, how about this feature? @diegotori
it's really helpful because video_player doesn't support different resolutions

@diegotori
Copy link
Collaborator

diegotori commented Jul 2, 2023

@merdweebner are you still actively working on this PR?

If so, please re-sync your changes with master's current changes.

@dejancencelj
Copy link

dejancencelj commented Feb 12, 2024

Any update?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for multiple resolutions
7 participants