Skip to content

[CP-stable]Fix Drawer prematurely snapping - #154720

Merged
auto-submit[bot] merged 1 commit into
flutter:flutter-3.24-candidate.0from
flutteractionsbot:cp-stable-e1f0083c7840a3fe7751c62db958455fe9f6566e
Sep 11, 2024
Merged

auto-submit[bot] merged 1 commit into
flutter:flutter-3.24-candidate.0from
flutteractionsbot:cp-stable-e1f0083c7840a3fe7751c62db958455fe9f6566e

Conversation

@flutteractionsbot

@flutteractionsbot flutteractionsbot commented Sep 6, 2024 •

Copy link
Copy Markdown
Contributor

Issue Link

#154722

Target

stable

Cherry pick PR Link

#154720

Changelog Description

The Drawer widget will snap completely open or shut after being dragged more than halfway.

Impacted Users

This affects everyone who has a Drawer widget in their mobile app.

Impact Description

Fortunately, the impact of this issue is limited to visual jank. But it will likely be widespread across app developers.

Workaround

There isn't a workaround for this.

Risk

low (This fix was merged over 2 weeks ago.)

Test Coverage

This fix is covered by the test added in the linked pull request.

Validation Steps

The fix can be verified by opening a mobile Flutter app and dragging a Drawer open or shut.

This regression is due to the fact that `animationController.isAnimating` and `animationController.status.isAnimating` sometimes return opposite values.

This PR implements & tests the bugfix, and also adds documentation to explain this behavior.

<br>

fixes flutter#153851

cc @azeunkn0wn @huycozy
@flutteractionsbot flutteractionsbot added the cp: review Cherry-picks in the review queue label Sep 6, 2024
@flutteractionsbot

This comment was marked as resolved.

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. a: animation Animation APIs p: material_ui material_ui package in flutter/packages labels Sep 6, 2024

@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.

Let's do it!

@itsjustkevin

Copy link
Copy Markdown
Contributor

@justinmc or @Piinks could you take a look at this cherry-pick?

@Piinks Piinks 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 all!

@itsjustkevin itsjustkevin added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 11, 2024
@auto-submit
auto-submit Bot merged commit 98ba471 into flutter:flutter-3.24-candidate.0 Sep 11, 2024
auto-submit Bot pushed a commit that referenced this pull request Sep 11, 2024
# Flutter stable 3.24.3 Framework

## Scheduled Cherrypicks
- #154380
- #154720
- #154944
@nate-thegrate nate-thegrate added cp: merged Cherry-pick has been merged to the release branch. and removed cp: review Cherry-picks in the review queue labels Sep 11, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 13, 2024
Gustl22 pushed a commit to Gustl22/flutter that referenced this pull request Nov 13, 2024
@amaralphp amaralphp mentioned this pull request Mar 27, 2025
9 tasks
@flutteractionsbot
flutteractionsbot deleted the cp-stable-e1f0083c7840a3fe7751c62db958455fe9f6566e branch July 20, 2026 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: animation Animation APIs autosubmit Merge PR when tree becomes green via auto submit App cp: merged Cherry-pick has been merged to the release branch. 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.

4 participants