Skip to content
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

Resolved #5Code cleaning and #1 Improved the UI #7

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Ayush-Khandelwal-007
Copy link

@Ayush-Khandelwal-007 Ayush-Khandelwal-007 commented Oct 3, 2020

#1

UI improvement

Whole UI was a mess earlier. A proper interface is created with change in position of elements and change of size and appearance of icons.
#5

Cleaning the code

Also there were many typos like use of class instead of className which don't created any deployment issue right now but might create in future.
Screenshot 2020-10-04 at 2 05 59 AM
Screenshot 2020-10-04 at 2 05 53 AM

@Ayush-Khandelwal-007
Copy link
Author

Also, it's a request if you are not willing to merge it then allow it for hacktoberfest by adding appropriate label.
Hoping for a positive response.

@Ayush-Khandelwal-007 Ayush-Khandelwal-007 changed the title Resolved #2 and #1 Resolved #5Code cleaning and #1 Improved the UI Oct 3, 2020
@Ishaan28malik
Copy link
Owner

@Ayush-Khandelwal-007 good job

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants