-
-
Notifications
You must be signed in to change notification settings - Fork 189
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
Fix login and signup page styling #1255
Fix login and signup page styling #1255
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you suveshmoza! for creating this pull request and contributing to Dummygram! 💗
The maintainers will review this Pull Request and provide feedback as soon as possible! 😇
We appreciate your patience and contribution, Keep up the great work! 😀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there suveshmoza!, congratulations on your first pull request :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor changes required. Will fix them :)
src/pages/Signup/index.jsx
Outdated
@@ -298,9 +298,9 @@ const SignupScreen = () => { | |||
handleSignInWithFacebook={(e) => | |||
signInWithOAuth(e, enqueueSnackbar, navigate, false) | |||
} | |||
have_acct_question={"Already have an account?"} | |||
have_acct_question={"Don't have an account?"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think perhaps this doesn't make sense, as we'd definitely want to sign in as an alternative in signup page. Also we'd want to navigate to signin from signup.
src/index.css
Outdated
@@ -88,10 +88,11 @@ body { | |||
} | |||
|
|||
.navbar_wrapper { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this class is of no use now as we removed it, we should remove it from here as well
Thanks a lot for your contribution to dummygram! :) |
Kudos, SonarCloud Quality Gate passed!
|
This PR addresses #1003.
I have fixed the text color in the dark mode and is now visible. Also the margin issue has been fixed and sign up form styling is fixed and looks exactly like login form.