-
Notifications
You must be signed in to change notification settings - Fork 84
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 details #70
base: master
Are you sure you want to change the base?
Added details #70
Conversation
Is it working fine? |
Nope @atulyadav745! The changes you have made are commented. Moreover, add the links in the PR description above as well :) |
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.
Do the required changes for proper submission!
Though, we could still review your tasks based on the (commented) link you have provided.
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 @atulyadav745, Good work! Clean code and simple UI. However, there are a few scopes of improvements like the responsiveness of the web app, and color choices. Also, your task 3 (Giving hints to users) is not working as expected.
You can try to implement task 2 as well, this would be helpful for you. Also, it will help you to explore other options to restart the game rather than reloading the page.
CSOC Task 1 Submission
Track
Task Completed
Hangman
Calculator
Submission
Repo
APK|Web Site