-
-
Notifications
You must be signed in to change notification settings - Fork 240
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
Fix TextField type prop for antd (closes #345) #346
Conversation
Also, I'm not quite sure how I would fully test this locally by importing my version of the library into a sample react app. Right now I'm purely going by the jest test. @radekmie how do you test new changes locally in a react app? |
@@ -15,6 +15,7 @@ const Text = props => | |||
placeholder={props.placeholder} | |||
ref={props.inputRef} | |||
value={props.value} | |||
type={props.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.
Could you keep it ordered?
Either by linking it to an existing app (symlink or |
Codecov Report
@@ Coverage Diff @@
## master #346 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 156 156
Lines 1301 1301
=====================================
Hits 1301 1301
Continue to review full report at Codecov.
|
I'll release it this weekend. |
Should fix #345
@radekmie Let me know if this PR is okay.
I didn't see a need to touch
filterDOMProps
especially since its being used by all other style versions. It seems it was just a missed prop.