-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added transition for the instruction box #255
Conversation
…ction This pull request addresses issue Rakesh9100#219 by introducing a Drag & Drop feature, providing users with a seamless and interactive way to upload content. Additionally, the Upload Section has undergone a redesign to improve its visual appeal and usability.
✅ Deploy Preview for click-the-edible-game ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@Rakesh9100 The PR is ready for a review and merge. Kindly review it and provide your feedback |
No need to change the UI. You have to just add transitions while opening and closing the box. Cross button added is OK. In addition if you want, you can work on making it translucent for good looking but without changing the UI @antilneeraj |
Hey @Rakesh9100 I have applied the animation to the modal without changing any UI Component. Please Review the PR and share your feedback |
The 2nd and 3rd screens instruction box is not appearing on clicking the Instruction button, pls fix @antilneeraj |
Hey @Rakesh9100, Please review the PR as i have fixed the modal issue and kindly provide your feedback. |
Good, Merged 🎉 @antilneeraj |
Fixes Issue🛠️
Description👨💻
Added Transition to the instruction box when clicked on the instruction button and also added a cross button to close the modal.
Type of change📄
How this has been tested✅
I have tested the changes Locally using alpha, beta & unit testing.
Checklist✅
Screenshots/GIF📷