-
Notifications
You must be signed in to change notification settings - Fork 149
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
Updated UI, fixed bugs and refactored code #45
Conversation
ac935ca
to
bea3c91
Compare
Fixed the issue #44 here itself |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @ItsAdityaKSingh, This PR still doesn't reflect the desired changes. In fact, UI had no issues earlier, but now it has (adding the screenshot below for reference). Maybe run gh pr checkout 45 or make sure you've pulled the changes locally and then try to test your changes.
Thanks for the PR @ItsAdityaKSingh , Merged! 🎉 |
This PR fixes issues:
Along with the UI improvements and minor bug fixes, I have minutely refactored the code for proper readability and understanding, added commas(,) at a few places, fixed sizing issues for all the containers and buttons, and also tried to improve the overall interface. Any suggestions for improvements are most welcome!