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

Allow Chewie controls to be positioned to allow for a larger safe area. #676

Merged
merged 5 commits into from
Nov 25, 2022

Conversation

jweidner-mbible
Copy link
Contributor

No description provided.

@diegotori
Copy link
Collaborator

diegotori commented Oct 19, 2022

LGTM so far. There might be a couple of nits, but that's it.

Once it passes automated checks, then it'll get merged into master.

lib/src/chewie_player.dart Outdated Show resolved Hide resolved
lib/src/chewie_player.dart Outdated Show resolved Hide resolved
@diegotori
Copy link
Collaborator

@jweidner-mbible please re-sync your change with master. Thanks.

@jweidner-mbible
Copy link
Contributor Author

@diegotori I just re-synced my change with master.

@caitlynbell399
Copy link
Contributor

@diegotori Any idea on when this might get merged into master? :)

@diegotori
Copy link
Collaborator

@caitlynbell399 rerunning the lint and format CI tasks. If they're green, then they'll look good to me

@diegotori
Copy link
Collaborator

@jweidner-mbible looks like the CI checks failed. Please address them, or else I won't be able to merge.

Thanks.

@caitlynbell399
Copy link
Contributor

@diegotori Committed a fix for the issue. Thank you for your help with this :)

@diegotori
Copy link
Collaborator

LGTM. Merging.

@diegotori diegotori merged commit 65391c4 into fluttercommunity:master Nov 25, 2022
diegotori added a commit that referenced this pull request Nov 25, 2022
lg8294 added a commit to lg8294/chewie that referenced this pull request Jan 3, 2023
* master:
  Version 1.3.6. Addresses PRs fluttercommunity#676 and fluttercommunity#681.
  fix default value for minimum safe area
  Cleaned up the Github Workflows YAML.
  Fixed remaining Dart Analysis issues after switching to flutter_lints.
  Example app is now using flutter_lints.
  Library is now using flutter_lints.
  Changed documentation as requested
  Use EdgeInsets.zero
  Allow Chewie controls to be positioned to allow for a larger safe area.
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.

3 participants