Skip to content

Change the definition of MainAxisSize - #5931

Merged
abarth merged 1 commit into
flutter:masterfrom
abarth:new_main_axis_size
Sep 19, 2016
Merged

abarth merged 1 commit into
flutter:masterfrom
abarth:new_main_axis_size

Conversation

@abarth

@abarth abarth commented Sep 19, 2016

Copy link
Copy Markdown
Contributor

This property now applies only to the free space in the flex layout (i.e.,
minimize or maximize the amount of free space). Previously, the flexible
children were always allocated a size of zero when MainAxisSize was min. Now
they're allocated the same size that would be if the MainAxisSize was max.

@abarth

abarth commented Sep 19, 2016

Copy link
Copy Markdown
Contributor Author

@Hixie

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe mention the constraints minWidth here?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

similarly here

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe mention that a tight flexible child is going to make this get essentially ignored?

@Hixie

Hixie commented Sep 19, 2016

Copy link
Copy Markdown
Contributor

LGTM

This property now applies only to the free space in the flex layout (i.e.,
minimize or maximize the amount of free space). Previously, the flexible
children were always allocated a size of zero when MainAxisSize was min. Now
they're allocated the same size that would be if the MainAxisSize was max.
@abarth
abarth merged commit e04135e into flutter:master Sep 19, 2016
@abarth
abarth deleted the new_main_axis_size branch September 19, 2016 22:05
engine-flutter-autoroll pushed a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 2, 2018
flutter-team-archive/engine@a76054f...3b66f20

git log a76054f..3b66f20 --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-02 [email protected] Don't drop MotionEvents with unknown tool type. (flutter#5931)
2018-08-02 37626415+[email protected] Roll src/third_party/skia 64cc576b1fa7..578ef2847b72 (20 commits) (flutter#5930)
2018-08-02 [email protected] Roll Dart to b04def964c428ada007cca7ef6b4936001db965d (flutter#5928)
2018-08-02 [email protected] Add an explicit `-[FlutterViewController init]` implementation (flutter#5924)
2018-08-02 37626415+[email protected] Roll src/third_party/skia e43024a5bab7..64cc576b1fa7 (1 commits) (flutter#5926)
2018-08-01 37626415+[email protected] Roll src/third_party/skia ed8bc196bd56..e43024a5bab7 (1 commits) (flutter#5925)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 3, 2018
flutter-team-archive/engine@4893b07...3b66f20

git log 4893b07..3b66f20 --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-02 [email protected] Don't drop MotionEvents with unknown tool type. (flutter#5931)
2018-08-02 37626415+[email protected] Roll src/third_party/skia 64cc576b1fa7..578ef2847b72 (20 commits) (flutter#5930)
2018-08-02 [email protected] Roll Dart to b04def964c428ada007cca7ef6b4936001db965d (flutter#5928)
2018-08-02 [email protected] Add an explicit `-[FlutterViewController init]` implementation (flutter#5924)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants