-
Notifications
You must be signed in to change notification settings - Fork 601
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
feat: Fixing Login Option #1143
Conversation
✅ Deploy Preview for taupe-cendol-f7e2bb ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@apu52 Please review my PR. |
@apu52 Review this PR once. |
@itsekta please review this PR once. |
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.
@AshmitaBarthwal it's good
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.
@AshmitaBarthwal but why this unnecessary commits present in this PR , as it does not have any linking with this...Please try to delete all of the unnecessary commit otherwise I can't merge this PR.
& I would like to suggest u to make different branches for different pr
Okay I will delete extra things |
@apu52 Please check the above changes. |
@apu52 i have deleted the unnecessary. |
@apu52 Please review the PR. |
Title and Issue number
Title : Fixing Login Option
Issue No. : #1087
Code Stack : CSS
Close #1087
Description
Added background according to light and dark mode as per website's color code to the buttons.
Video/Screenshots (mandatory)
BEFORE IN LIGHT MODE:
BEFORE IN DARK MODE:
AFTER IN LIGHT MODE:
AFTER IN DARK MODE:
Type of PR
Checklist:
contributing.md
file before contributingAdditional context:
##Are you contributing under any Open-source programme?
GSSOC'24
VSOC'24