-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
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
[Button] Expose disableTouchRipple #6234
[Button] Expose disableTouchRipple #6234
Conversation
0457788
to
40b92c8
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.
Thanks for opening that second PR.
src/FlatButton/FlatButton.js
Outdated
@@ -36,6 +36,10 @@ class FlatButton extends Component { | |||
*/ | |||
className: PropTypes.string, | |||
/** | |||
* If true, the element's ripple effect will be disabled. | |||
*/ | |||
disableTouchRipple: React.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.
Prop types declarations should be sorted alphabetically react/sort-prop-types
- The enhanced button wrapper exposes an option to disable touch ripples. This is the case with the IconButton and should be persisted to the FlatButton. - In addition to exposing the prop, the documentation page will be automatically updated with the new prop for Icon Button as well, since the documentation for the props is pulled strictly from the FlatButton component
40b92c8
to
90ff625
Compare
- The enhanced button wrapper exposes the option to disable touch ripples. This is the case with the FlatButton component and should be exposed on RaisedButton for consistency. - In addition to exposing the prop, the documentation page will automatically be updated with the newly defined prop from the RaisedButton component
90ff625
to
956b29a
Compare
@oshalygin Thanks, I have merged this PR and close the other one as RaisedButton & FlatButton are deeply linked. An atomic PR update sounds better. |
This PR directly references:
#6232
disable touch ripples. This is the case with the FlatButton
component and should be exposed on RaisedButton for consistency.
automatically with the newly defined prop from the RaisedButton Component.
Please wait to merge this until the previous PR is merged and this PR is rebased.