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
TextField - new outline preset #3066
Conversation
…nto feat/TextField_outline_preset # Conflicts: # demo/src/screens/componentScreens/TextFieldScreen.tsx # src/components/textField/types.ts # src/components/textField/usePreset.ts
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.
Looks good
Approved, see my comment on the example screen, it will help testing the component better
@@ -74,6 +80,7 @@ export default class TextFieldScreen extends Component { | |||
/> | |||
<TextField | |||
placeholder="Placeholder" | |||
placeholderTextColor={Colors.$textNeutralLight} |
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.
Why are you setting a different placeholderColor here?
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 was just testing new token. Removed
this.setState({preset: index === 0 ? 'underline' : 'outline'}); | ||
}; | ||
|
||
getDynamicFieldStyle = (context, props) => { |
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.
This behavior can be achieved by passing a color object with all states, any reason you are doing it like this?
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.
We don't have a prop to control the underline color with the ColorType... What are you referring to?
@ethanshar I added another change to the demo's configuration file to allow overriding |
Description
TextField - new outline preset
based on: #3065
Changelog
TextField - new outline preset
Additional info
ticket 4172