-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work lista: error messageError messages from the Flutter frameworkError messages from the Flutter frameworkf: scrollingViewports, list views, slivers, etc.Viewports, list views, slivers, etc.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.perf: speedPerformance issues related to (mostly rendering) speedPerformance issues related to (mostly rendering) speed
Description
See more context in #103068
The framework has at least one place where we recommend using shrinkWrap: true in an assertion error.
We should stop doing this. We should instead recommend using some kind of custom sliver scheme, or just using some kind of non-scrollable widget like a Wrap or Row/Column. ShrinkWrapping can come with additional performance penalities on build/raster that get harder to reason about and test.
goderbauer, jonahwilliams, pingbird and Piinks
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work lista: error messageError messages from the Flutter frameworkError messages from the Flutter frameworkf: scrollingViewports, list views, slivers, etc.Viewports, list views, slivers, etc.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.perf: speedPerformance issues related to (mostly rendering) speedPerformance issues related to (mostly rendering) speed