material-components-flutter-adaptive icon indicating copy to clipboard operation
material-components-flutter-adaptive copied to clipboard

fix child loss in AdaptiveColumn & use flutter >2.4

Open huang12zheng opened this issue 4 years ago • 3 comments

  1. use flutter >2.4
  2. fix #27

huang12zheng avatar Nov 09 '21 10:11 huang12zheng

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Nov 09 '21 10:11 google-cla[bot]

@googlebot I signed it!

huang12zheng avatar Nov 09 '21 10:11 huang12zheng

Thanks for this PR. A couple things: API changes should be broken out to a separate PR and documented in the changelog, so anyone currently using brightness or textTheme knows what to do. Second, it would be nice to have a test to validate that this works.

guidezpl avatar Nov 09 '21 11:11 guidezpl

Thank you for your contribution. I'm going to close this PR for now since there are outstanding comments, just to get this off our PR review queue. Please don't hesitate to submit a new PR at https://github.com/material-foundation/flutter-packages if you have the time to address the review comments. Thanks!

guidezpl avatar Feb 02 '23 09:02 guidezpl