Skip to content

Don't use shrink wrap in our samples #116237

Description

@goderbauer

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

Activity

  1. added
    frameworkflutter/packages/flutter repository. See also f: labels.
    f: scrollingViewports, list views, slivers, etc.
    on Nov 30, 2022
  2. goderbauer commented on Nov 30, 2022

    @goderbauer
    MemberAuthor

    cc @Piinks @jonahwilliams

    (I removed shrinkWrapping from all of our other samples in #116236, this is the last remaining one.)

  3. added
    P2Important issues not at the top of the work list
    c: tech-debtTechnical debt, code quality, testing, etc.
    on Nov 30, 2022
  4. github-actions commented on Jul 25, 2023

    @github-actions

    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 -v and a minimal reproduction of the issue.

  5. locked as resolved and limited conversation to collaborators on Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

P2Important issues not at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.d: examplesSample code and demosf: scrollingViewports, list views, slivers, etc.frameworkflutter/packages/flutter repository. See also f: labels.team-frameworkOwned by Framework teamtriaged-frameworkTriaged by Framework team

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions