-
Notifications
You must be signed in to change notification settings - Fork 27.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reland "Update ExpansionTile to support Material 3 & add an example" #121212
Reland "Update ExpansionTile to support Material 3 & add an example" #121212
Conversation
Looks like there are some failures to fix... |
b8d59d9
to
a472dc8
Compare
Apologies, fixed an unintended example comment. However, the current failures are related to #121242 (I also see them on my other PRs) |
Aha, I see. #121242 is a P0, so I trust it will be resolved soon. Using onSurfaceVariant for the iconColor instead of onSurface is OK with me. Doing that, for consistency with ListTile, has been suggested by other developers too. |
a472dc8
to
3083488
Compare
@TahaTesser - unfortunately one more internal change must land before this PR can. Fingers crossed, we may be good to go tomorrow. |
auto label is removed for flutter/flutter, pr: 121212, due to - Please get at least one approved review if you are already a member or two member reviews if you are not a member before re-applying this label. Reviewers: If you left a comment approving, please use the "approve" review action instead. |
auto label is removed for flutter/flutter, pr: 121212, due to Validations Fail. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
reland #119712 with feedback
fixes #119627
@HansMuller
I updated
collapsedIconColor
withonSurfaceVariant
instead ofonSurface
, to be consistent with theListTile
widget which I updated in a recent PR #120444flutter/packages/flutter/lib/src/material/list_tile.dart
Line 1596 in 4c25587
Pre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.