-
Notifications
You must be signed in to change notification settings - Fork 151
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
UPDATE: ContactInput, NEW: Asterisk at label #337
Conversation
9e28d09
to
395cfda
Compare
e952ab3
to
39a81cd
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.
Just this question 👍
@@ -4,6 +4,7 @@ export type Props = { | |||
children: React$Node, | |||
filled?: boolean, | |||
disabled?: boolean, | |||
required?: boolean, | |||
}; |
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.
Should be typing for FormLabel
strict too?
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 think it's not necessary, Why do you need 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.
Because when someone will import FormLabel
when needed to use compact InputField
, then it's necessary to be strict in flow type, otherwise the person will be able to pass eg. className
.
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 think if someone want to use FormLabel they should read documentation firstly how to use. If there will be more questions about this, we can update, so for now it is OK.
bb78172
to
b3002aa
Compare
b3002aa
to
1689086
Compare
1689086
to
3e8da13
Compare
This Pull Request meets the following criteria: