Skip to content

Add optional param to override selectionOverlay widget for CupertinoDatePicker #79559

Closed

Description

@lecicdjuro

CupertinoDatePicker doesn't have optional param for selectionOverlay Widget, so it can't be overridden or changed.
This class should have the same option as CupertinoPicker, so we can change this overlay widget.

@DanielEdrisian I'm using the latest stable version

[✓] Flutter (Channel stable, 2.0.3, on macOS 11.2.3 20D91 darwin-x64, locale en-RS)
• Flutter version 2.0.3 at /Users/lecicdjuro/flutter
• Framework revision 4d7946a (2 weeks ago), 2021-03-18 17:24:33 -0700
• Engine revision 3459eb2
• Dart version 2.12.2

Activity

  1. added
    in triagePresently being triaged by the triage team
    p: cupertino_uicupertino_ui package in flutter/packages
    frameworkflutter/packages/flutter repository. See also f: labels.
    c: proposalA detailed proposal for a change to Flutter
    and removed
    in triagePresently being triaged by the triage team
    on Apr 5, 2021
  2. TahaTesser commented on Apr 5, 2021

    @TahaTesser
    Contributor

    Related #68509

  3. Jaaco commented on Oct 11, 2023

    @Jaaco

    Any updates on when this will be added?

  4. TahaTesser commented on Oct 11, 2023

    @TahaTesser
    Contributor

    Any updates on when this will be added?

    PRs are welcome, feel free to create one. Happy to review.

  5. varunkamanibosc commented on Feb 6, 2024

    @varunkamanibosc
    Contributor

    Hi @TahaTesser I would like to work on this feature. Can you please guide me?

  6. varunkamanibosc commented on Feb 7, 2024

    @varunkamanibosc
    Contributor

    @TahaTesser I went through the code. Here are my observations.

    We have 3 columns. Each column has its own overlay. We have 3 constants. _startSelectionOverlay, _centerSelectionOverlay, _endSelectionOverlay. All 3 overlays are using CupertinoPickerDefaultSelectionOverlay. We are managing the border radius for each overlay.

    For this new functionality, Should I add 3 new properties called: startSelectionOverlay, centerSelectionOverlay, endSelectionOverlayto theCupertinoDatePicker?

  7. varunkamanibosc commented on Feb 8, 2024

    @varunkamanibosc
    Contributor

    @TahaTesser I have opened a PR #143079. Can you please take a look?

  8. victorsanni commented on Aug 23, 2024

    @victorsanni
    Contributor

    This should be fixed by #143079.

  9. added
    r: fixedIssue is closed as already fixed in a newer version
    on Aug 23, 2024
  10. github-actions commented on Sep 6, 2024

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

  11. locked as resolved and limited conversation to collaborators on Sep 6, 2024
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

    c: new featureNothing broken; request for a new capabilityc: proposalA detailed proposal for a change to Flutterf: date/time pickerDate or time picker widgetsframeworkflutter/packages/flutter repository. See also f: labels.p: cupertino_uicupertino_ui package in flutter/packagesr: fixedIssue is closed as already fixed in a newer versionteam-designOwned by Design Languages teamtriaged-designTriaged by Design Languages team

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions