-
-
Notifications
You must be signed in to change notification settings - Fork 93
#1693 - Migrated newUserForm.jsx to MUI #2012
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
base: development
Are you sure you want to change the base?
#1693 - Migrated newUserForm.jsx to MUI #2012
Conversation
I fixed the heading style on "Welcome!". |
Hi @bconti123 - thanks for submitting this PR! A couple of small bits of feedback:
Once this is complete, can you please add some updated screenshots for review? |
I’ve updated the MUI styles and screenshots. This PR is now ready for review. |
Thanks so much for the quick turnaround on these fixes @bconti123! Everything is looking good! One final minor fix - can you please change the text on these two lines from grey to black text so that the typographic hierarchy remains consistent for the user? |
Fixes #1693
What changes did you make and why did you make them?
Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)
Visuals before changes are applied
Visuals after changes are applied