Repository navigation
Don't use shrink wrap in our samples #116237
Copy link
Copy link
Closed
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.d: examplesSample code and demosSample code and demosf: 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.team-frameworkOwned by Framework teamOwned by Framework teamtriaged-frameworkTriaged by Framework teamTriaged by Framework team
Description
Activity
- addedframeworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.f: scrollingViewports, list views, slivers, etc.Viewports, list views, slivers, etc.d: examplesSample code and demosSample code and demos
on Nov 30, 2022 (I removed shrinkWrapping from all of our other samples in #116236, this is the last remaining one.)
- addedP2Important issues not at the top of the work listImportant issues not at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.
on Nov 30, 2022 - addedteam-frameworkOwned by Framework teamOwned by Framework teamtriaged-frameworkTriaged by Framework teamTriaged by Framework team
on Jul 8, 2023 - moved this from Todo to Done in Scrolling Performance Pitfalls
on Jul 11, 2023 - added a commit that references this issue
on Jul 11, 2023 This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of
flutter doctor -vand a minimal reproduction of the issue.- locked as resolved and limited conversation to collaborators
on Jul 25, 2023 - added a commit that references this issue
on Jun 12, 2026
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.d: examplesSample code and demosSample code and demosf: 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.team-frameworkOwned by Framework teamOwned by Framework teamtriaged-frameworkTriaged by Framework teamTriaged by Framework team
Type
Projects
- StatusShow more project fieldsDone
https://github.com/flutter/flutter/blob/fa063eb4ce4140dec9a887c3130239c68f7b713e/examples/api/lib/material/refresh_indicator/refresh_indicator.1.dart uses shrinkWrap, but really it shouldn't. Unfortunately, it is currently hard to achieve this effect without shrinkWrapping. We should implement whatever additional Slivers are needed to not require shrinkWrapping here (e.g. SliverDecoration).