aayushai / TranspilerX

Code converter from one programming language to another.
https://transpiler-x.vercel.app
MIT License
1 stars 7 forks source link

#1 Fix : Loading Spinner #6

Closed AnshMehta24 closed 9 hours ago

AnshMehta24 commented 10 hours ago

Issue: #1

The loading spinner was occupying the entire screen and did not align well with the theme, impacting user experience.

Fix:

Replaced the previous loading spinner with ClipLoader from the react-spinners library.

The ClipLoader is now only visible within the Output Editor, ensuring the spinner is limited to the relevant area.

vercel[bot] commented 10 hours ago

@AnshMehta24 is attempting to deploy a commit to the Aayush Pandey's projects Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] commented 9 hours ago

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

Name Status Preview Comments Updated (UTC)
transpiler-x ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 11, 2024 0:40am
AnshMehta24 commented 9 hours ago

Preview:

https://github.com/user-attachments/assets/cf1be22a-bc19-4890-b098-2aff49d7bfab

it's my first contribution, so I'd appreciate any feedback.

aayushai commented 9 hours ago

Good job keep contributing to open-source.

AnshMehta24 commented 8 hours ago

hey @aayushai , i think you have not merged my whole code (Its becasue of Merge Conflicts). you can go through my code and check the preview i have commented it's looking good instead of whole screen gone black..

aayushai commented 8 hours ago

Whole screen is not going black, only the output field has spinner contained. check this Link

AnshMehta24 commented 8 hours ago

ohh I see. you just commited that Remaining code. because from this pr you have changed only 2 lines that 's why im confused. now i have checked that you have added that remaining lines of code.

aayushai commented 8 hours ago

Yeah, while merging conflicts it made an error so i had to do it.

AnshMehta24 commented 8 hours ago

got it!!