Skip to content

Fix Scaffold extend body - #157441

Merged
auto-submit[bot] merged 7 commits into
flutter:masterfrom
yiiim:fix_scaffold_extend_body
Oct 27, 2024
Merged

auto-submit[bot] merged 7 commits into
flutter:masterfrom
yiiim:fix_scaffold_extend_body

Conversation

@yiiim

@yiiim yiiim commented Oct 23, 2024

Copy link
Copy Markdown
Member

Fixes: #157316

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages labels Oct 23, 2024
@Piinks
Piinks requested a review from nate-thegrate October 23, 2024 18:05

@nate-thegrate nate-thegrate left a comment

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.

Thanks for fixing this!

Comment thread packages/flutter/test/material/scaffold_test.dart
Comment thread packages/flutter/lib/src/material/scaffold.dart Outdated
Comment thread packages/flutter/lib/src/material/scaffold.dart
Comment thread packages/flutter/test/material/scaffold_test.dart

@nate-thegrate nate-thegrate left a comment

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.

LGTM, thanks again for the fix!

Feel free to land this one if current diffs look good to you :)

@yiiim

yiiim commented Oct 25, 2024

Copy link
Copy Markdown
Member Author

What is making Google testing unhappy?

@nate-thegrate

Copy link
Copy Markdown
Contributor

What is making Google testing unhappy?

I'm not a Googler so idk, sorry 😔

Maybe you could see if a rebase fixes it? If not, we can get somebody else to step in.

@nate-thegrate

Copy link
Copy Markdown
Contributor

success

woohoo!

@nate-thegrate nate-thegrate added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 27, 2024
@auto-submit
auto-submit Bot merged commit 04bcfe9 into flutter:master Oct 27, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 27, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 27, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 28, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 28, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 28, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 28, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 28, 2024
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Oct 28, 2024
flutter/flutter@5a11904...3fb95c7

2024-10-28 [email protected] Roll Flutter Engine from bf5bfeebf89f to 23b2c6d3b3b1 (3 revisions) (flutter/flutter#157736)
2024-10-28 [email protected] Avoid labeling 'context' PRs as 'text-input' (flutter/flutter#157650)
2024-10-28 [email protected] Add test for `restorable_route_future.0.dart` (flutter/flutter#157708)
2024-10-28 [email protected] Update .ci.yaml have android sdk be 35v1 (flutter/flutter#157621)
2024-10-28 [email protected] Roll Flutter Engine from 2522789c4199 to bf5bfeebf89f (1 revision) (flutter/flutter#157713)
2024-10-28 [email protected] Add test for `restoration_mixin.0.dart` (flutter/flutter#157709)
2024-10-28 [email protected] Add test for `navigator.restorable_push_replacement.0.dart` (flutter/flutter#157704)
2024-10-28 [email protected] Add test for `focus_node.unfocus.0.dart` (flutter/flutter#157673)
2024-10-28 [email protected] Roll Flutter Engine from 9226b089d634 to 2522789c4199 (2 revisions) (flutter/flutter#157710)
2024-10-28 [email protected] Add tests for `navigator_state.restorable_push.0.dart` (flutter/flutter#157667)
2024-10-28 [email protected] Roll Flutter Engine from 10e50bd42c42 to 9226b089d634 (1 revision) (flutter/flutter#157702)
2024-10-28 [email protected] Add test for `navigator_state.restorable_push_replacement.0.dart` (flutter/flutter#157668)
2024-10-28 [email protected] Remove unsafe_html lint rule (flutter/flutter#157531)
2024-10-27 [email protected] Roll Flutter Engine from 3162aaad0086 to 10e50bd42c42 (1 revision) (flutter/flutter#157690)
2024-10-27 [email protected] Fix Scaffold extend body (flutter/flutter#157441)
2024-10-26 [email protected] Roll Flutter Engine from c9b8ac96f6ce to 3162aaad0086 (3 revisions) (flutter/flutter#157681)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@maxfrees

maxfrees commented Nov 8, 2024 •

Copy link
Copy Markdown

@yiiim @nate-thegrate @Piinks when will it be released, or how can I use it in advance

@yiiim

yiiim commented Nov 8, 2024

Copy link
Copy Markdown
Member Author

@yiiim @nate-thegrate @Piinks when will it be released, or how can I use it in advance

Currently, it can only be used on the master channel. You can switch to the master channel using flutter channel master, but it is highly discouraged to do so as the master channel is usually very unstable.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 12, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 7, 2025
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
flutter/flutter@5a11904...3fb95c7

2024-10-28 [email protected] Roll Flutter Engine from bf5bfeebf89f to 23b2c6d3b3b1 (3 revisions) (flutter/flutter#157736)
2024-10-28 [email protected] Avoid labeling 'context' PRs as 'text-input' (flutter/flutter#157650)
2024-10-28 [email protected] Add test for `restorable_route_future.0.dart` (flutter/flutter#157708)
2024-10-28 [email protected] Update .ci.yaml have android sdk be 35v1 (flutter/flutter#157621)
2024-10-28 [email protected] Roll Flutter Engine from 2522789c4199 to bf5bfeebf89f (1 revision) (flutter/flutter#157713)
2024-10-28 [email protected] Add test for `restoration_mixin.0.dart` (flutter/flutter#157709)
2024-10-28 [email protected] Add test for `navigator.restorable_push_replacement.0.dart` (flutter/flutter#157704)
2024-10-28 [email protected] Add test for `focus_node.unfocus.0.dart` (flutter/flutter#157673)
2024-10-28 [email protected] Roll Flutter Engine from 9226b089d634 to 2522789c4199 (2 revisions) (flutter/flutter#157710)
2024-10-28 [email protected] Add tests for `navigator_state.restorable_push.0.dart` (flutter/flutter#157667)
2024-10-28 [email protected] Roll Flutter Engine from 10e50bd42c42 to 9226b089d634 (1 revision) (flutter/flutter#157702)
2024-10-28 [email protected] Add test for `navigator_state.restorable_push_replacement.0.dart` (flutter/flutter#157668)
2024-10-28 [email protected] Remove unsafe_html lint rule (flutter/flutter#157531)
2024-10-27 [email protected] Roll Flutter Engine from 3162aaad0086 to 10e50bd42c42 (1 revision) (flutter/flutter#157690)
2024-10-27 [email protected] Fix Scaffold extend body (flutter/flutter#157441)
2024-10-26 [email protected] Roll Flutter Engine from c9b8ac96f6ce to 3162aaad0086 (3 revisions) (flutter/flutter#157681)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
flutter/flutter@5a11904...3fb95c7

2024-10-28 [email protected] Roll Flutter Engine from bf5bfeebf89f to 23b2c6d3b3b1 (3 revisions) (flutter/flutter#157736)
2024-10-28 [email protected] Avoid labeling 'context' PRs as 'text-input' (flutter/flutter#157650)
2024-10-28 [email protected] Add test for `restorable_route_future.0.dart` (flutter/flutter#157708)
2024-10-28 [email protected] Update .ci.yaml have android sdk be 35v1 (flutter/flutter#157621)
2024-10-28 [email protected] Roll Flutter Engine from 2522789c4199 to bf5bfeebf89f (1 revision) (flutter/flutter#157713)
2024-10-28 [email protected] Add test for `restoration_mixin.0.dart` (flutter/flutter#157709)
2024-10-28 [email protected] Add test for `navigator.restorable_push_replacement.0.dart` (flutter/flutter#157704)
2024-10-28 [email protected] Add test for `focus_node.unfocus.0.dart` (flutter/flutter#157673)
2024-10-28 [email protected] Roll Flutter Engine from 9226b089d634 to 2522789c4199 (2 revisions) (flutter/flutter#157710)
2024-10-28 [email protected] Add tests for `navigator_state.restorable_push.0.dart` (flutter/flutter#157667)
2024-10-28 [email protected] Roll Flutter Engine from 10e50bd42c42 to 9226b089d634 (1 revision) (flutter/flutter#157702)
2024-10-28 [email protected] Add test for `navigator_state.restorable_push_replacement.0.dart` (flutter/flutter#157668)
2024-10-28 [email protected] Remove unsafe_html lint rule (flutter/flutter#157531)
2024-10-27 [email protected] Roll Flutter Engine from 3162aaad0086 to 10e50bd42c42 (1 revision) (flutter/flutter#157690)
2024-10-27 [email protected] Fix Scaffold extend body (flutter/flutter#157441)
2024-10-26 [email protected] Roll Flutter Engine from c9b8ac96f6ce to 3162aaad0086 (3 revisions) (flutter/flutter#157681)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When using extendBody and the card & input box appear at the same time, the screen will flash white

3 participants