Skip to content

Conversation

@Pramod-Munnoli
Copy link
Contributor

Hi @venisha-kalola 👋

I noticed that after PR #73 was merged, the index.html file for the theme switcher was accidentally removed.
I’ve created a follow-up PR (fix/issue-73-restore-html) to restore the missing file.

This PR only restores the HTML file and does not include any other changes.
Thank you for reviewing it! 🙏

@vercel
Copy link

vercel bot commented Oct 26, 2025

@Pramod-Munnoli is attempting to deploy a commit to the hrishikesh-dalal's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Pramod-Munnoli
Copy link
Contributor Author

Hi @@venisha-kalola 👋

This PR is part of Hacktoberfest 2025. Could you please add the label hacktoberfest-accepted so it counts toward the event?

Thank you very much for your time and review! 🙏

@venisha-kalola
Copy link
Contributor

add screenshots of your implementation

@Pramod-Munnoli
Copy link
Contributor Author

Hi @venisha-kalola 👋

Here’s the full UI screenshot of the restored index.html, showing the working theme switcher interface and popup notifications:

🖼️ Combined Screenshot (4-in-1)
imgonline-com-ua-twotoone-Z6ss5YpcL2EHb0p

This image includes:

  • Color applied confirmation message
  • Error message when saving palette without a name
  • Palette deleted confirmation message
  • Shared link popup message

🖼️ Full implementation screenshot:
them-switcher-UI

These confirm that the Theme Switcher HTML page and its features are working correctly after restoring the missing file.
Thank you for reviewing!

@Hrishikesh-Dalal Hrishikesh-Dalal merged commit c3112a8 into commitra:main Oct 27, 2025
1 of 2 checks passed
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.

3 participants