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

Port to material-ui v1 beta #28

Closed

Conversation

chaitan94
Copy link

This should hopefully fix #25

Cheers,
chaitan94

@leMaik
Copy link
Member

leMaik commented Nov 2, 2017

It's a very good start, thank you very much! ❤️
I'll continue the work on your branch soon.

@leMaik leMaik self-assigned this Nov 2, 2017
@leMaik leMaik mentioned this pull request Nov 2, 2017
@m2mathew
Copy link

m2mathew commented Mar 6, 2018

@leMaik Have you had a chance to review this PR? I was about to port this over to v1 for one of our CEA projects, but realized that @chaitan94 had already started the work. Thanks!

@leMaik
Copy link
Member

leMaik commented Mar 6, 2018

@m2mathew This PR is now outdated (plenty of breaking changes in material-ui) and not done yet. If you want, we'd really like to see this work to be continued! 💯

Otherwise, I'll port it as soon as I've got time. There is a project where we might need this component later.

@m2mathew
Copy link

m2mathew commented Mar 8, 2018

Yeah, I was about 15 versions behind on MUI beta. API is much different, but feels great to use. I might have time to dig back into this soon!

@leMaik
Copy link
Member

leMaik commented Mar 23, 2018

I'll close this in favor of #35

@leMaik leMaik closed this Mar 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Port to material-ui 1.0.0-beta
3 participants