Skip to content

Revert DraggableScrollableSheet controller changes#112293

Merged
auto-submit[bot] merged 2 commits intoflutter:masterfrom
CaseyHillers:revert-draggable-changes
Sep 23, 2022
Merged

Revert DraggableScrollableSheet controller changes#112293
auto-submit[bot] merged 2 commits intoflutter:masterfrom
CaseyHillers:revert-draggable-changes

Conversation

@CaseyHillers
Copy link
Contributor

This broke some use cases causing several assets to get squashed. An example was a triangle icon becoming a flat line.

@flutter-dashboard flutter-dashboard bot added f: scrolling Viewports, list views, slivers, etc. framework flutter/packages/flutter repository. See also f: labels. labels Sep 23, 2022
@moffatman
Copy link
Contributor

Weird. Do you have an example of the failure, so I can ensure it's fixed when relanding?

Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

RSLGTM

@CaseyHillers CaseyHillers added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 23, 2022
@CaseyHillers
Copy link
Contributor Author

Weird. Do you have an example of the failure, so I can ensure it's fixed when relanding?

@chunhtai or @HansMuller will be able to reproduce a more complete example, but there was an instance of an svg icon being squished.

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 f: scrolling Viewports, list views, slivers, etc. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants