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

[feature]: Add images to the game #81

Merged
merged 1 commit into from
Oct 9, 2022

Conversation

Rafajrg21
Copy link
Contributor

@Rafajrg21 Rafajrg21 commented Oct 8, 2022

What does this PR do?

This PR makes it so instead of just letters, it shows 16 cards from a collection of images, some styling was modified as well to have a better presentation of such images. This new functionality can be extended by creating different themes for the game, adding replay value.

The images are smash bros related, because it was the first thing I thought of.

Related PRs and Issues

This solves issue #75

Screenshots of changes made to the webapp.

A new game looks like this

Already in the game

Have you updated the README.md file?

  • yes
  • no

Have you made any big breakthrough change?

  • yes
  • no

Is the code beautified?

  • yes
  • no

Is the code clear to understand with proper variable naming and comments?

  • yes
  • no

@Aatmaj-Zephyr
Copy link
Owner

Thanks for the work!
Please fill the pull request template with appropriate checklist and screenshots so I can approve and merge it

@Rafajrg21
Copy link
Contributor Author

@Aatmaj-Zephyr are those screenshots enough? Should I add anything else?

@Aatmaj-Zephyr
Copy link
Owner

Thanks

@Aatmaj-Zephyr Aatmaj-Zephyr merged commit fbb6b0a into Aatmaj-Zephyr:main Oct 9, 2022
Copy link
Owner

@Aatmaj-Zephyr Aatmaj-Zephyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved after merging

jma26 added a commit to jma26/Basic-memory-game that referenced this pull request Oct 12, 2022
jma26 added a commit to jma26/Basic-memory-game that referenced this pull request Oct 12, 2022
jma26 added a commit to jma26/Basic-memory-game that referenced this pull request Oct 12, 2022
jma26 added a commit to jma26/Basic-memory-game that referenced this pull request Oct 12, 2022
jma26 added a commit to jma26/Basic-memory-game that referenced this pull request Oct 12, 2022
Aatmaj-Zephyr pushed a commit that referenced this pull request Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants