Skip to content

[FlatButton][RaisedButton] Fix label validation for 0 (number data-type)#4269

Closed
mrdivyansh wants to merge 2 commits intomui:masterfrom
mrdivyansh:labelValidationFix
Closed

[FlatButton][RaisedButton] Fix label validation for 0 (number data-type)#4269
mrdivyansh wants to merge 2 commits intomui:masterfrom
mrdivyansh:labelValidationFix

Conversation

@mrdivyansh
Copy link
Copy Markdown

FlatButton throw an error when label is zero (numeric data-type). This is special case, so we should handle it in validation function.

@mbrookes
Copy link
Copy Markdown
Member

@mrdivyansh Could you fix RaisedButton too please?

@mbrookes mbrookes added PR: Needs Revision type: bug It doesn't behave as expected. labels May 16, 2016
@mrdivyansh
Copy link
Copy Markdown
Author

@mbrookes PR ready :)

@mbrookes mbrookes changed the title [FlatButton] Fix label validation for 0 (number data-type) [FlatButton][RaisedButton] Fix label validation for 0 (number data-type) May 17, 2016
@mbrookes
Copy link
Copy Markdown
Member

Thanks, if you could just change the PR titles to:

"[FlatButton] Fix label validation for 0 (number data-type)"

and

"[RaisedButton] Fix label validation for 0 (number data-type)"

respectively, then we're good to go. Thanks!

@mrdivyansh
Copy link
Copy Markdown
Author

I have created two new pull requests. #4284 & #4285

@mrdivyansh mrdivyansh closed this May 17, 2016
@mrdivyansh mrdivyansh deleted the labelValidationFix branch May 17, 2016 14:46
@mbrookes
Copy link
Copy Markdown
Member

This one was fine. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants