Skip to content

Vertical Stepper should have scrollController field #61207

Closed
#128814
Closed
Vertical Stepper should have scrollController field#61207
#128814
Labels
P3Issues that are less important to the Flutter projectc: proposalA detailed proposal for a change to Flutterf: scrollingViewports, list views, slivers, etc.frameworkflutter/packages/flutter repository. See also f: labels.p: material_uimaterial_ui package in flutter/packagesr: fixedIssue is closed as already fixed in a newer version

Description

@cpboyd
cpboyd
opened on Jul 10, 2020

Use case

For other components that are intended to wrap Scrollable widgets, like DraggableScrollableSheet, it's necessary to specify the scrollController.

I'm currently using this to specify a scrollController, however, it adds an extra, otherwise unnecessary SingleChildScrollView:

             SingleChildScrollView(
                controller: scrollController,
                child: Stepper(
                  physics: ClampingScrollPhysics(),
                  ...
                ),
              ),

Proposal

Ideally, it'd be nice to simply use this instead:

             Stepper(
                scrollController: scrollController,
                ...
             ),

Activity

  1. darshankawar
    added
    f: scrollingViewports, list views, slivers, etc.
    frameworkflutter/packages/flutter repository. See also f: labels.
    c: proposalA detailed proposal for a change to Flutter
    on Jul 10, 2020
  2. goderbauer
    added
    p: material_uimaterial_ui package in flutter/packages
    on Jul 15, 2020
  3. pedromassangocode
    added
    passed first triage
    on Jan 5, 2021
  4. Piinks
    added
    P3Issues that are less important to the Flutter project
    on Feb 21, 2023

  5. SwaroopSambhayya commented on Jun 4, 2023

    @SwaroopSambhayya
    SwaroopSambhayya
    on Jun 4, 2023
    Contributor

    Hi @darshankawar can i contribute to this?

  6. darshankawar commented on Jun 5, 2023

    @darshankawar
    darshankawar
    on Jun 5, 2023
    Member

    @SwaroopSambhayya Sure. You may checkout contributing guidelines https://github.com/flutter/flutter/blob/master/CONTRIBUTING.md

  7. HansMuller
    closed this as completedin #128814on Jun 20, 2023
  8. fluttermirroringbot
    added a commit that references this issue on Jun 20, 2023

    Adding ScrollController support for Stepper widget (#128814)

    80935ad
  9. darshankawar
    added
    r: fixedIssue is closed as already fixed in a newer version
    on Jun 21, 2023

  10. github-actions commented on Jul 5, 2023

    @github-actions
    github-actionsbot
    on Jul 5, 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 -v and a minimal reproduction of the issue.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    P3Issues that are less important to the Flutter projectc: proposalA detailed proposal for a change to Flutterf: scrollingViewports, list views, slivers, etc.frameworkflutter/packages/flutter repository. See also f: labels.p: material_uimaterial_ui package in flutter/packagesr: fixedIssue is closed as already fixed in a newer version

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions