Skip to content

showCupertinoDialog should support barrierDismissible #45744

Closed

Description

@mehmetf

Internal: https://b.corp.google.com/issues/145094605

In showCupertinoDialog it is not supported to enabled to dismiss the dialog on outside tap. It seems that this might be the normal iOS behaviour, but I couldn't find any documentation that it is not allowed to support it.

Activity

  1. added
    frameworkflutter/packages/flutter repository. See also f: labels.
    on Dec 4, 2019
  2. alectogeek commented on Jan 2, 2020

    @alectogeek

    We realy need this feature!

  3. LongCatIsLooong commented on Jan 2, 2020

    @LongCatIsLooong
    Contributor

    I couldn't find anything that says this is not allowed, but from https://developer.apple.com/design/human-interface-guidelines/ios/views/alerts/ it shouldn't be encouraged either, as barrierDismissible = true makes it a lot easier for users to dismiss something important by mistake:

    Minimize alerts. Alerts disrupt the user experience and should only be used in important situations like confirming purchases and destructive actions (such as deletions), or notifying people about problems. The infrequency of alerts helps ensure that people take them seriously. Ensure that each alert offers critical information and useful choices.

    Sounds reasonable to add the parameter but set the default to false.

  4. tottomotto commented on Jan 29, 2020

    @tottomotto
    Contributor

    @xster,can you please share your thoughts on this.
    I Implemented the change (defaulting to false) and I can make a PR if it's ok.

  5. xster commented on Feb 4, 2020

    @xster
    Member

    Making it configurable (but defaulted to false) seems reasonable to me.

  6. github-actions commented on Aug 21, 2021

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

  7. locked as resolved and limited conversation to collaborators on Aug 21, 2021
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

    customer: googleVarious Google teamsframeworkflutter/packages/flutter repository. See also f: labels.p: cupertino_uicupertino_ui package in flutter/packages

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions