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
Add onKeyDown to TextField and TextArea #303
Add onKeyDown to TextField and TextArea #303
Conversation
Deploy preview for gestalt ready! Built with commit 4d83201 |
Generated by 🚫 dangerJS |
1a2afb4
to
fe93cab
Compare
75f4993
to
96e3ca2
Compare
@@ -44,16 +44,24 @@ card( | |||
}, | |||
{ | |||
name: 'onBlur', | |||
type: '({ event: SyntheticFocusEvent<>, value: string }) => void', | |||
type: |
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.
These are going to cause Flow errors when bumping in pinboard - prepare yourselves, winter is coming
handleChange = (event: SyntheticInputEvent<HTMLTextAreaElement>) => { | ||
this.props.onChange({ | ||
event, | ||
value: event.currentTarget.value, |
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.
are we sure this is the same, event.target.value
and event.currentTarget.value
? i know there was once a currentTarget
target
change in pinboard and it caused a rollback/hotfix
No description provided.