-
Notifications
You must be signed in to change notification settings - Fork 3.4k
[flutter_adaptive_scaffold] Adds additional slot animation parameters #7411
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
Conversation
a3445fb
to
35e46ea
Compare
@jamiewest can you pitch on how the out animation is supposed to work? |
34e9b88
to
af460bb
Compare
872133c
to
07ca445
Compare
@Renzo-Olivares can you check this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gspencergoog done. |
64594f7
to
16136d8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM but would it be considered a breaking change since duration
is being removed?
16136d8
to
f1a3e06
Compare
@Renzo-Olivares I've updated the version number and changelog. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but deferring to @gspencergoog regarding removing duration
being a breaking change. If it is then it should be stated in the changelog.
e3c7251
to
7004764
Compare
dcaab41
to
3312839
Compare
flutter/packages@330581f...df88c81 2024-09-14 [email protected] [webview_flutter] Improve flaky scroll tests (flutter/packages#7621) 2024-09-13 [email protected] Bump deps (flutter/packages#7357) 2024-09-13 [email protected] [flutter_adaptive_scaffold] Use improved MediaQuery methods (flutter/packages#7565) 2024-09-13 [email protected] [many] Update example android apps to target SDK level 34. (flutter/packages#7587) 2024-09-13 [email protected] [pigeon] adds support for non nullable types in collections (flutter/packages#7547) 2024-09-13 [email protected] [flutter_adaptive_scaffold] Adds additional slot animation parameters (flutter/packages#7411) 2024-09-13 [email protected] Roll Flutter from 303f222 to 2d30fe4 (21 revisions) (flutter/packages#7646) 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-flutter-autoroll Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: 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
Slots can currently be animated in and out of view, but there is not an ability to change the duration or curves of these animations. This PR adds additional parameters that are passed to the underlying AnimatedSwitcher when switching slots in and out of view.
List which issues are fixed by this PR. You must list at least one issue.
#6957
Pre-launch Checklist
dart format
.)[shared_preferences]
pubspec.yaml
with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.CHANGELOG.md
to add a description of the change, following repository CHANGELOG style, or this PR is exempt from CHANGELOG changes.///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.