PriyansuMaurya / AI-Fusion

AI Fusion is your ultimate destination for streamlined access to a curated collection of powerful AI tools and prompts.
https://aifusion.vercel.app/
MIT License
13 stars 8 forks source link

fix: toggle button background in light mode #31

Closed AlimurtuzaCodes closed 1 year ago

AlimurtuzaCodes commented 1 year ago

PR

Issue Number: #30

Fixed background color to green in light mode of toggle button.

vercel[bot] commented 1 year ago

Someone is attempting to deploy a commit to a Personal Account owned by @PriyansuMaurya on Vercel.

@PriyansuMaurya first needs to authorize it.

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated (UTC)
aifusion ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 22, 2023 11:08am
PriyansuMaurya commented 1 year ago

Current:

image

Your changes:

image

If you look closely, your changes do not seem to be consistent with the overall UI.

I have checked all the available options.

image

But I think old one is very consistent with the UI. So, I would say go with the old one and keep the size to 30

image

Ameya02 commented 1 year ago

@CYBWEBALI please make the changes given by @PriyansuMaurya and update the PR

AlimurtuzaCodes commented 1 year ago

Hence there are no changes you can close this pr

rajkaranm commented 1 year ago

Hey Thanks for contribution but this issue can be solved by just adding

className="text-[color:var(--primary-color)]"

To Navbar component in line 22, this line add class of color to --primary-color which green by tailwind. Hope you understood and just change color don't change fontSize of the icon.

AlimurtuzaCodes commented 1 year ago

Hey Thanks for contribution but this issue can be solved by just adding

className="text-[color:var(--primary-color)]"

To Navbar component in line 22, this line add class of color to --primary-color which green by tailwind. Hope you understood and just change color don't change fontSize of the icon.

Current Output after adding this line: image

Expected Ouput: image

PriyansuMaurya commented 1 year ago

Hence, there are no changes. You can close this PR.

Thanks for your patience, @CYBWEBALI.

I just checked myself, and now I understand that this problem cannot be solved with the current icon, BsToggleOff.

Check out this repository, I think it could solve this issue: https://github.com/clari/react-ios-switch.

AlimurtuzaCodes commented 1 year ago

I just did a small fix,I just rotated the BsToggleOn and I have achieved what we wanted. image image image

If it is Ok then I will create New PR for this

PriyansuMaurya commented 1 year ago

Great! Create a new PR.

AlimurtuzaCodes commented 1 year ago

Updated This PR Only.

PriyansuMaurya commented 1 year ago

@CYBWEBALI , You nailed this pull request! Keep up the good work.