Skip to content

Conversation

@ShaunByrne-UniSA
Copy link
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. f: cupertino flutter/packages/flutter/cupertino repository 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
Copy link
Contributor

@MitchellGoodwin MitchellGoodwin left a comment

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.

Copy link
Contributor

@Piinks Piinks left a comment

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
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
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 f: cupertino flutter/packages/flutter/cupertino repository framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CupertinoDatePicker - Non-selectable days are not greyed out.

3 participants