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

Functionality of hacker card completed. #24

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aadishJ01
Copy link

Polyglot JS Task Submission

  • Checkpoint 1: Compare the cards of hacker and player.

20pts

  • Checkpoint 2: Keep a track of the individual scores of the hacker and player.

20pts

  • Checkpoint 3: Deploy the code to GitHub Pages.

10pts

  • Extras: You can improve UI of the game ;).

Submission

Repo
Deployed Site

@shubhangi013
Copy link
Collaborator

@aadishJ01 Amazing work! 🎉
Loved the custom name functionality.

  • The prompt asks the user to enter their first name and even if the user enters more than one string, the name gets displayed perfectly with a mild error, the last name also gets lowercased because of the toLowerCase() method.
  • Also, if possible, work with the CSS and align the winning and losing messages for smaller devices.

@Divyansh013 Divyansh013 added the passed Great work label Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
passed Great work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants