Closed aysal04 closed 1 year ago
Yeah sorry. I deleted it accidentally. Currently I have made some changes to address issue #928 . Can I commit both the changes together here? Or shall a open a new PR?
Yeah sorry. I deleted it accidentally. Currently I have made some changes to address issue #928 . Can I commit both the changes together here? Or shall a open a new PR?
It is always recommended to work on different issues using separate branches in order to maintain a clean history.
Additionally, since the issue in question has not been assigned to you, you can proceed with working on it if the assigned person does not submit a pull request (PR) in a timely manner.
Just commit the changes for this pull request here and refrain from addressing any other issues in this pull request.
Thank you @aysal04 for taking out your valuable time in order to contribute to our project. Looking forward for more such amazing contributions :)
Thank you @aysal04 for taking out your valuable time in order to contribute to our project. Looking forward for more such amazing contributions :)
It was an amazing experience contributing for the first time to an open source. Thanks to my mentor for such an amazing guidance
Appreciate your contribution :)
It was really an amazing experience contributing for the first time to an open source. Thanks to you for being so supportive
🛠️ Fixes #976
Closes #976
👨💻 Changes proposed
I have added a toggle checkbox that allows us to switch between light and darkmodes. For this I have used basic java script to change the colors based on the status of the checkbox.
✅ Check List (Check all the applicable boxes)
📄 Note to reviewers
As the color of the numbers on the clock is fixed, we couldn't change them. However grey suits both the themes. Here is the preview of light and dark mode UI/UX designs of Analog clock.
📷 Screenshots