-
Notifications
You must be signed in to change notification settings - Fork 303
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
Changes committed #470
Changes committed #470
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.
Welcome to Our repository.🎊 Thank you so much for taking the time to point this out.
@Shivam250702 are you working? |
I have pushed the changes you required kindly check |
Resolve the conflicts. Pull all changes from origin and add a video or screenshot to demonstrate how the website works both in dark and light mode. |
Resolve the conflicts. Pull all recent changes from origin |
In light mode you can keep some light shade of green. Make sure all the content is visible clearly whatever colour you choose. |
Also check the placement of the mode option in small devices |
Once you are done raise anothe PR |
Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
#213
Description
[Please include a brief description of the changes or features added]
Added the Dark mode feature on Navbar
Type of PR
Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:
Additional context:
[Include any additional information or context that might be helpful for reviewers.]