-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
Support foregroundColor
for TextField
#453
Conversation
The CI failure is fixed by #454, which has been merged. |
TextField
TextField
foregroundColor
for TextField
Great, thanks! |
public var textFieldStyle: _AnyTextFieldStyle { subject.environment.textFieldStyle } | ||
public var foregroundColor: AnyColorBox.ResolvedValue? { | ||
guard let style = subject.environment._foregroundStyle else { return nil } | ||
var shape = _ShapeStyle_Shape( |
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.
I'm a bit confused why ShapeStyle
specifically is used here. Could you clarify that in code comments?
@carson-katri I'm thinking of tagging a new version in the next few days with support for |
I should be able to get this finished up either tonight or tomorrow. |
No rush, many thanks for your contributions! 🙂 |
Releasing 0.9.0 today, but I'm happy to tag 0.9.1 or whatever patch release would include this fix, when it's ready. |
17dfae7
to
554c0b8
Compare
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!
No description provided.