-
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
Migrate TextField to Material 3 #108366
Migrate TextField to Material 3 #108366
Conversation
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
editableText = tester.widget(find.byType(EditableText)); | ||
expect(editableText.style.color, setColor); | ||
|
||
// inherit: false causes nothing to be merged in from theme |
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.
NICE
String generate() => ''' | ||
// Generated version ${tokens["version"]} | ||
|
||
TextStyle _m3InputStyle(BuildContext context) => ${textStyle("md.comp.filled-text-field.label-text")}!; |
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.
Filled text fields and outlined text fields use the same text style? Same color?
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.
Yes they are the same
* Update input style * generate M3 text style * Add test
@hangyujin I tried the Affected widgets: (not including TextField) |
How do I get the "filled" text field style in material 3? Example on the left (Image from #103537) Edit: Nevermind, use |
fixed: #103537
Migrate TextField to Material 3.
The inputDecorator is already migrated to M3 with fillColor, boderColor, iconColor, label text style, supporting text style.
So this PR is just to migrate input text style because it's in TextField, not the InputDecorator.
Pre-launch Checklist
///
).