-
Notifications
You must be signed in to change notification settings - Fork 55
light/dark theme toggle #169
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
Conversation
|
@JiyaBatra is attempting to deploy a commit to the Harsh Yadav's projects Team on Vercel. A member of the Team first needs to authorize it. |
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 @JiyaBatra for your first PR to SaveBook!
We really appreciate your contribution 🙌
What happens next:
- 🔍 Maintainers will review your PR
- 🧪 Automated checks will run
- ✨ Feedback may be shared if needed
Please confirm your PR includes:
- ✔️ Clear summary of changes
- ✔️ Linked issue (e.g., Fixes #123)
- ✔️ Steps to test
- ✔️ Screenshots (for UI changes)
📘 Contribution Standards:
👉 https://github.com/HarshYadav152/SaveBook/blob/main/CONTRIBUTING.md
💬 Stay Connected with Our Community
🎯 Discord (Official Communication)
For formal discussions, code reviews, and project updates:
👉 https://discord.gg/eMTePUK3gJ
💚 WhatsApp (Friendly Community)
For informal chats, quick help, and building friendships with contributors:
👉 https://chat.whatsapp.com/I8GYXd3mHlDCC2iXhNGeqV
🌟 Our Philosophy: We value both professional collaboration (Discord) and personal connections (WhatsApp). Join both to get the complete SS-Capture community experience!
Thanks for helping improve SaveBook 🚀
Let's build something amazing together! 💪
|
@JiyaBatra Why your PR have so much files changes. |
Having any problem reach out to us. |
|
@HarshYadav152 Thanks for the feedback! Most of the changes are from working on Navbar.js, plus some updates in global.css and Tailwind CSS. The rest are files added/updated due to npm install and dependencies. |
keep node_modules folder name inside .gitignore |
|
@HarshYadav152 I don’t have internet right now, I’ll add it to .gitignore tomorrow or just add the three files, and sorry for this mistake. |


📌 Description
Resolved the light/dark theme toggle issue that was assigned (#106). Now, the theme switches correctly across all pages and components without breaking existing styles.
Fixes #106
🔧 Type of Change
Bug fix (non-breaking change which fixes an issue)
How Has This Been Tested?
Manually tested the theme toggle on all pages to ensure proper switching.
Verified that no component styling breaks during theme change.
Checklist:
Screenshot
Uploading SaveBook _ Your Personal Notebook & Cloud Note-Taking App - Google Chrome 2026-01-25 15-35-25.mp4…