Skip to content

Conversation

@HansMuller
Copy link
Contributor

For tiles with selected: true, if the tile's activeColor is null, the selected item's text color now defaults to ThemeData.toggleableActiveColor instead of ThemeData.accentColor. This removes the widget's accentColor dependency per #56918.

This is technically not a breaking change, because it doesn't break any existing tests. However the dependency had been documented.

@flutter-dashboard flutter-dashboard bot added f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. labels Feb 27, 2021
@google-cla google-cla bot added the cla: yes label Feb 27, 2021
Copy link
Contributor

@darrenaustin darrenaustin left a comment

Choose a reason for hiding this comment

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

LGTM

@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

@HansMuller
Copy link
Contributor Author

Note: there was one Google testing failure, which turned out to have been a flake.

@HansMuller HansMuller merged commit 4903640 into flutter:master Mar 1, 2021
@HansMuller HansMuller deleted the remove_accent_color_in_switch_list_tile branch March 1, 2021 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants