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

Well, there goes my entire weekend... ^^ (This is the correct one) #8

Open
wants to merge 66 commits into
base: main
Choose a base branch
from

Conversation

RandomPianist
Copy link

@RandomPianist RandomPianist commented Jun 20, 2022

THIS IS AN UPDATED VERSION OF MY PULL REQUEST

I saw this repo at saturday morning and I felt the urge of dive myself in async JavaScript.
Now it's Sunday 11pm (now Monday 6pm) in Brazil and I do not regret anything. Maybe I should eat something lol

Anyway, your application is superb, but I dare propose a few ameliorations.

Improvements

  • Deleted unuseful commented code. All comments in this version are documentation-relative.
  • Improved code presentation: spacing and identation remade.
  • Fixed a bug regarding the long exposure of the leader card.
  • Fixed typo in page title.
  • Fixed the style of end screen buttons.
  • Fixed notification read times.
  • Added sound effects to the game.

Changes

  • The game is now played in full screen.
  • As in the original game, passing the round now requires a countdown timer.
  • As in the original game, modifying cards in the deck now requires double-clicking (or pressing "Enter" key [still in alfa]).

Operations included with the keyboard:

"E" starts the game
"X" uses or modifies the leader card
"Q" closes the card explanation windows
"Space" passes the current round
"Enter" plays the cards
Arrows select cards on the carousel

Deleting in order to upload a new file tree
Deleting in order to upload a new file tree
Deleting in order to upload a new file tree
Deleting in order to upload a new file tree
Adds: 1 - a page for rules (by @romain-durban, author of version 3.0), 2 - an A.I. vs A.I. option (by @romain-durban, author of version 3.0), 3 - a statistics table, 4 - a "give up" button
Adds: 1 - a page for rules (by @romain-durban, author of version 3.0), 2 - an A.I. vs A.I. option (by @romain-durban, author of version 3.0), 3 - a whole new set of cards and mechanics (by @romain-durban, author of version 3.0), 4 - a statistics table, 5 - a "give up" button, 6 - new visuals for carousel.
Corrects: 1 - aesthetic values, 2 - unused code, 3 - bug in leader selection, 4 - bug when drawing a match, 5 - typo errors, 6 - bug when redrawing cards at the start of each match.
Cards in deck selector are now shown in alphabetical order
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.

3 participants