make light/dark mode theme persist on refresh

[KeithBrosch]

Description

Fixed the onClick behavior of the Dark/Light Mode toggle so that the theme persists on refresh.

Related Issues

[#1057]

Changes Proposed

Fixed the behavior of toggleTheme() within Sidebar.jsx to make use of our useTheme hook, and removed redundant lines of code within toggleTheme.

Now that we're properly using useTheme, we don't need to explicitly write things like:

htmlElement.classList.remove('dark');
setTheme('light');

because these changes happen dynamically for us already in other parts of the code.

Checklist

• I have read and followed the Contribution Guidelines.
• All new and existing tests passed.
• I have updated the documentation to reflect the changes I've made.
• My code follows the code style of this project.
• The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
dev-find	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 19, 2024 10:12pm

[github-actions]

Great job, @KeithBrosch! 🎉 Thank you for opening a pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

Soon the maintainers/owner will review it and provide you with feedback/suggestions.

[KeithBrosch]

There is another PR, #1061, that may technically fix this issue, but it doesn't make use of the useTheme hook, and instead just replicates it's functionality within toggleTheme.