Skip to content

Completely added dark mode functionality#256

Open
Tapesh-1308 wants to merge 13 commits intohereisSwapnil:mainfrom
Tapesh-1308:main
Open

Completely added dark mode functionality#256
Tapesh-1308 wants to merge 13 commits intohereisSwapnil:mainfrom
Tapesh-1308:main

Conversation

@Tapesh-1308
Copy link
Contributor

Description

Added dark mode for every page matching with the UI. Now users can toggle the theme.

  • every page made suitable for dark mode commit by committing for your easier understanding
  • at last added toggle functionality
  • created a theme context, improved settings page UI and enabled it to toggle theme.

Resolves multiple issues people raised:
#221 #113 #74 #38

Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

  • Have you adhered to the repository's defined coding convention rules?

Screenshots

image

Similarly, I have added dark mode for all pages.

Additional Notes/Comments

It took a lot of effort to completely switch to dark mode since there are many pages. Can you please label this PR as level 3😊


I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

@vercel
Copy link

vercel bot commented May 29, 2024

@Tapesh-1308 is attempting to deploy a commit to the swapnilsingh99's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job, @Tapesh-1308! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@Tapesh-1308
Copy link
Contributor Author

@hereisSwapnil Can you please quickly review it, as this PR is changing many files and delaying can cause a lot of merge conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant