Skip to content

Grey out non-selectable days in CupertinoDatePicker - #136181

Merged
auto-submit[bot] merged 1 commit into
flutter:masterfrom
ShaunByrne-UniSA:fix136179
Feb 5, 2024
Merged

auto-submit[bot] merged 1 commit into
flutter:masterfrom
ShaunByrne-UniSA:fix136179

Conversation

@ShaunByrne-UniSA

Copy link
Copy Markdown
Contributor

This PR changes the text color of non-selectable days in the CupertinoDatePicker to match non-selectable months and years.

Current:
Screenshot_1696829290

New:
Screenshot_1696830232

Fixes #136179

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. p: cupertino_ui cupertino_ui package in flutter/packages labels Oct 9, 2023
@ShaunByrne-UniSA
ShaunByrne-UniSA force-pushed the fix136179 branch 2 times, most recently from c38e6cf to 75e8ff1 Compare January 17, 2024 04:47

@MitchellGoodwin MitchellGoodwin 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! Thank you for putting this together.

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

Hey @ShaunByrne-UniSA welcome! Thanks for contributing, LGTM!

@Piinks Piinks added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 31, 2024
@MitchellGoodwin

Copy link
Copy Markdown
Contributor

@ShaunByrne-UniSA would you be able to rebase this PR? That would likely get the testing unstuck.

@auto-submit
auto-submit Bot merged commit c21fbab into flutter:master Feb 5, 2024
@ShaunByrne-UniSA
ShaunByrne-UniSA deleted the fix136179 branch February 5, 2024 23:50
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 6, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 6, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 6, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 6, 2024
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Feb 6, 2024
Manual roll requested by [email protected]

flutter/flutter@0b5cd50...e6ba809

2024-02-06 [email protected] Roll Packages from ae3494d to 1a5a7ce (2 revisions) (flutter/flutter#142985)
2024-02-06 [email protected] TextField context menu should fade on scroll on mobile devices (flutter/flutter#138313)
2024-02-06 [email protected] Roll Flutter Engine from 9c1b6c98f7f0 to 808886312e2b (1 revision) (flutter/flutter#142959)
2024-02-06 [email protected] Roll Flutter Engine from 9bd98bc2fcdf to 9c1b6c98f7f0 (4 revisions) (flutter/flutter#142954)
2024-02-06 [email protected] Roll Flutter Engine from f34c658b9600 to 9bd98bc2fcdf (1 revision) (flutter/flutter#142950)
2024-02-05 [email protected] Grey out non-selectable days in CupertinoDatePicker (flutter/flutter#136181)
2024-02-05 49699333+dependabot[bot]@users.noreply.github.com Bump actions/upload-artifact from 4.3.0 to 4.3.1 (flutter/flutter#142944)
2024-02-05 [email protected] Run examples_smoke_test on Linux (flutter/flutter#142736)
2024-02-05 [email protected] Update AGP version validation code to support KGP and kotlin build files. (flutter/flutter#142357)
2024-02-05 [email protected] Fixed test in language_version_test.dart that failed when shuffling, â�¦ (flutter/flutter#142904)
2024-02-05 [email protected] Move Mac_build_test flutter_gallery__transition_perf_e2e_ios to staging (flutter/flutter#142918)
2024-02-05 [email protected] Roll Packages from d37fb0a to ae3494d (3 revisions) (flutter/flutter#142915)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 14, 2024
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
)

Manual roll requested by [email protected]

flutter/flutter@0b5cd50...e6ba809

2024-02-06 [email protected] Roll Packages from ae3494d to 1a5a7ce (2 revisions) (flutter/flutter#142985)
2024-02-06 [email protected] TextField context menu should fade on scroll on mobile devices (flutter/flutter#138313)
2024-02-06 [email protected] Roll Flutter Engine from 9c1b6c98f7f0 to 808886312e2b (1 revision) (flutter/flutter#142959)
2024-02-06 [email protected] Roll Flutter Engine from 9bd98bc2fcdf to 9c1b6c98f7f0 (4 revisions) (flutter/flutter#142954)
2024-02-06 [email protected] Roll Flutter Engine from f34c658b9600 to 9bd98bc2fcdf (1 revision) (flutter/flutter#142950)
2024-02-05 [email protected] Grey out non-selectable days in CupertinoDatePicker (flutter/flutter#136181)
2024-02-05 49699333+dependabot[bot]@users.noreply.github.com Bump actions/upload-artifact from 4.3.0 to 4.3.1 (flutter/flutter#142944)
2024-02-05 [email protected] Run examples_smoke_test on Linux (flutter/flutter#142736)
2024-02-05 [email protected] Update AGP version validation code to support KGP and kotlin build files. (flutter/flutter#142357)
2024-02-05 [email protected] Fixed test in language_version_test.dart that failed when shuffling, â�¦ (flutter/flutter#142904)
2024-02-05 [email protected] Move Mac_build_test flutter_gallery__transition_perf_e2e_ios to staging (flutter/flutter#142918)
2024-02-05 [email protected] Roll Packages from d37fb0a to ae3494d (3 revisions) (flutter/flutter#142915)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
)

Manual roll requested by [email protected]

flutter/flutter@0b5cd50...e6ba809

2024-02-06 [email protected] Roll Packages from 3e58f35 to 1c32385 (2 revisions) (flutter/flutter#142985)
2024-02-06 [email protected] TextField context menu should fade on scroll on mobile devices (flutter/flutter#138313)
2024-02-06 [email protected] Roll Flutter Engine from 9c1b6c98f7f0 to 808886312e2b (1 revision) (flutter/flutter#142959)
2024-02-06 [email protected] Roll Flutter Engine from 9bd98bc2fcdf to 9c1b6c98f7f0 (4 revisions) (flutter/flutter#142954)
2024-02-06 [email protected] Roll Flutter Engine from f34c658b9600 to 9bd98bc2fcdf (1 revision) (flutter/flutter#142950)
2024-02-05 [email protected] Grey out non-selectable days in CupertinoDatePicker (flutter/flutter#136181)
2024-02-05 49699333+dependabot[bot]@users.noreply.github.com Bump actions/upload-artifact from 4.3.0 to 4.3.1 (flutter/flutter#142944)
2024-02-05 [email protected] Run examples_smoke_test on Linux (flutter/flutter#142736)
2024-02-05 [email protected] Update AGP version validation code to support KGP and kotlin build files. (flutter/flutter#142357)
2024-02-05 [email protected] Fixed test in language_version_test.dart that failed when shuffling, â�¦ (flutter/flutter#142904)
2024-02-05 [email protected] Move Mac_build_test flutter_gallery__transition_perf_e2e_ios to staging (flutter/flutter#142918)
2024-02-05 [email protected] Roll Packages from 1888fb4 to 3e58f35 (3 revisions) (flutter/flutter#142915)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels. p: cupertino_ui cupertino_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CupertinoDatePicker - Non-selectable days are not greyed out.

3 participants