Skip to content

CupertinoAlertDialog should not create ScrollController on every build, if null values are passed in constructor. - #133918

Merged
polina-c merged 3 commits into
flutter:masterfrom
polina-c:ScrollController-fix
Sep 5, 2023
Merged

CupertinoAlertDialog should not create ScrollController on every build, if null values are passed in constructor.#133918
polina-c merged 3 commits into
flutter:masterfrom
polina-c:ScrollController-fix

Conversation

@polina-c

Copy link
Copy Markdown
Contributor

Fixes#133912

@github-actionsgithub-actionsBot added framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages p: cupertino_ui cupertino_ui package in flutter/packages labels Sep 2, 2023

@chunhtaichunhtai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@polina-c
polina-c merged commit 7cdf314 into flutter:masterSep 5, 2023
@polina-c
polina-c deleted the ScrollController-fix branch September 5, 2023 20:01
polina-c added a commit that referenced this pull request Sep 5, 2023
…ery build, if null values are passed in constructor. (#133918)"
This reverts commit 7cdf314.
auto-submitBot pushed a commit that referenced this pull request Sep 5, 2023
…ery build, if null values are passed in constructor." (#134071)
Reverts #133918 as it causes build failures.
@polina-c
polina-c restored the ScrollController-fix branch September 5, 2023 21:22
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 7, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 15, 2023
chunhtai pushed a commit to chunhtai/packages that referenced this pull request Jun 12, 2026
…ery build, if null values are passed in constructor." (#134071)
Reverts flutter/flutter#133918 as it causes build failures.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frameworkflutter/packages/flutter repository. See also f: labels.p: cupertino_uicupertino_ui package in flutter/packagesp: material_uimaterial_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CupertinoAlertDialog should not create ScrollController on every build, if null values are passed in constructor.

2 participants

@polina-c@chunhtai