Skip to content
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

[typescript] Remove xxxClassName props from type declarations #10644

Merged
merged 1 commit into from Mar 14, 2018
Merged

[typescript] Remove xxxClassName props from type declarations #10644

merged 1 commit into from Mar 14, 2018

Conversation

lukePeavey
Copy link
Contributor

Fixes #10636. I removed all the props that were changed in #10600. Let me know if there is anything I need to add/change.

@oliviertassinari oliviertassinari merged commit d1219fc into mui:v1-beta Mar 14, 2018
@oliviertassinari
Copy link
Member

@lukePeavey Perfect :)

@ianschmitz
Copy link
Contributor

@oliviertassinari shouldn't the class keys have been updated as part of this PR?

@oliviertassinari
Copy link
Member

@ianschmitz Do we miss some?

@ianschmitz
Copy link
Contributor

ianschmitz commented Mar 14, 2018

Hmm i may be misunderstanding the usage.

It looks like:

  • ButtonBase is missing keyboardFocused
  • TextField is missing helperText and label

@lukePeavey
Copy link
Contributor Author

ButtonBase is missing keyboardFocused
TextField is missing helperText and label

@ianschmitz Your right, those are missing from classKey. @oliviertassinari, I can submit another PR to fix that.

@oliviertassinari
Copy link
Member

oliviertassinari commented Mar 14, 2018

I can submit another PR to fix that.

@lukePeavey It would be great :)!

@lukePeavey lukePeavey deleted the remove-xxx-classname-props-from-typescript branch March 15, 2018 01:33
@oliviertassinari
Copy link
Member

TextField is missing helperText and label

@ianschmitz These keys don't exist. Given the TextField is a simple wrapper piping the properties between the FormControl, InputLabel, Input, etc. It doesn't come with a withStyles wrapper.

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

Successfully merging this pull request may close these issues.

None yet

3 participants