Skip to content

Conversation

@thkim1011
Copy link
Contributor

@thkim1011 thkim1011 commented Apr 6, 2023

Implements a new Sliver that constrains the child sliver's cross axis extent.

Eventually, we'll also add a flex value to SliverPhysicalParentData which must be explicitly set to 0 by this widget in order to override the SliverCrossAxisGroup's default behavior to make slivers expand.

This is prerequisite work for #123862.
Fixes #120896

@flutter-dashboard flutter-dashboard bot added f: scrolling Viewports, list views, slivers, etc. framework flutter/packages/flutter repository. See also f: labels. labels Apr 6, 2023
@thkim1011 thkim1011 requested a review from Piinks April 6, 2023 17:47
@thkim1011 thkim1011 requested a review from Piinks April 7, 2023 16:53
@flutter-dashboard flutter-dashboard bot added d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos documentation c: contributor-productivity Team-specific productivity, code health, technical debt. labels Apr 7, 2023
@thkim1011 thkim1011 requested a review from Piinks April 7, 2023 20:53
Copy link
Contributor

@Piinks Piinks left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 11, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 12, 2023
exaby73 pushed a commit to exaby73/flutter_nevercode that referenced this pull request Apr 17, 2023
thkim1011 added a commit that referenced this pull request Apr 20, 2023
auto-submit bot pushed a commit that referenced this pull request Apr 20, 2023
Going to make some changes to the implementation so I'll revert in the meantime.

Reverts #124337
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 c: contributor-productivity Team-specific productivity, code health, technical debt. c: new feature Nothing broken; request for a new capability d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos 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.

Add cross axis to SliverPrototypeExtentList

2 participants