-
Notifications
You must be signed in to change notification settings - Fork 8
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
Replace Reach UI components with something maintained #1378
Comments
Hey @millianapia, left some comments in your commits =) |
@fernandolucchesi have now refactored and fixed the UI changes after your recommendations :) ready to be reviewed again |
Hi @millianapia , left a couple more comments :P |
@fernandolucchesi fixed some things, added a couple of TODO and an explanation for the styling choices |
Nice! @millianapia I've replied to the thread about the stylings |
Looks good! |
Hi @meols @NickHaggerty1 , we have replaced a library that we used for a few components. There should not be any difference from your part, only from a code perspective. So the testcase is to check if these components are working like before:
|
Works like before as far as I can see 👍 Thanks @millianapia |
Reach UI is currently not maintained, they suggest using one of the following libraries instead:
The following components needs to be refactored to a new library:
The text was updated successfully, but these errors were encountered: