Skip to content

Fix default icon color constants reversed brightness documentation - #130231

Merged
auto-submit[bot] merged 2 commits into
flutter:masterfrom
rydmike:fix-doc-for-default-icon-color-constants
Jul 10, 2023
Merged

auto-submit[bot] merged 2 commits into
flutter:masterfrom
rydmike:fix-doc-for-default-icon-color-constants

Conversation

@rydmike

@rydmike rydmike commented Jul 9, 2023

Copy link
Copy Markdown
Contributor

Fixes the reversed default icon color constants brightness documentation for kDefaultIconLightColor and kDefaultIconDarkColor.

FIX: #130230

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.

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages labels Jul 9, 2023

@TahaTesser TahaTesser 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!

@TahaTesser TahaTesser added autosubmit Merge PR when tree becomes green via auto submit App and removed autosubmit Merge PR when tree becomes green via auto submit App labels Jul 10, 2023
@TahaTesser

Copy link
Copy Markdown
Contributor

This'll need a secondary review from @HansMuller for the autosubmit to pass.

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

Nice catch, thanks for the fix!
LGTM

@HansMuller HansMuller added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 10, 2023
@auto-submit
auto-submit Bot merged commit b828846 into flutter:master Jul 10, 2023
@rydmike
rydmike deleted the fix-doc-for-default-icon-color-constants branch July 10, 2023 15:44
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 12, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 12, 2023
gbtb16 pushed a commit to gbtb16/flutter that referenced this pull request Nov 6, 2023
…lutter#130231)

Fixes the reversed default icon color constants brightness documentation for `kDefaultIconLightColor` and `kDefaultIconDarkColor`.

FIX: flutter#130230
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: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation error: The brightness values for kDefaultIconLightColor and kDefaultIconDarkColor are reversed

3 participants