-
Notifications
You must be signed in to change notification settings - Fork 394
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
[components] Respect disabled prop on input components and cleanup #286
Conversation
kristofferjs
commented
Oct 17, 2017
- Removing hasFocus, error etc that is no longer in use (yet)
- fixing isSelected on searchable select
- Proper use of :disabled pseudo class where available
…other pros. Removing hasFocus etc that is no longer in use
@@ -31,7 +31,7 @@ export default class DefaultTextInput extends React.PureComponent { | |||
onBlur: PropTypes.func, | |||
isClearable: PropTypes.bool, | |||
isSelected: PropTypes.bool, | |||
isDisabled: PropTypes.bool, | |||
disabled: PropTypes.bool, |
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.
Renaming this is a potential breaking change. Have you searched for usages across other files in the repo?
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.
Found one use on SlugInput on form-builder. Updated
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.
Thanks!
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.
LGTM!