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

Tabithan Ho FTBC6 #61

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

Tabithan Ho FTBC6 #61

wants to merge 6 commits into from

Conversation

tabbiho
Copy link

@tabbiho tabbiho commented Jan 13, 2022

Please fill out the survey before submitting the pull request. Thanks!

🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀

How many hours did you spend on this assignment?
4 hours

Please fill in one error and/or error message you received while working on this assignment.
Encountered some errors when trying to clearTimeout for purpose of extending the delay before a function is run and stopping the overall game timer when the game is won. However, learned that a global variable to hold the setTimeout function is required for use in another function.

What part of the assignment did you spend the most time on?
Figuring out how some of page's elements appear and disappear as the player progresses in the game, and deciding which are deliberate triggers and which are timed.

Comfort Level (1-5):
5

Completeness Level (1-5):
5

What did you think of this deliverable?
It looks and behave like an actual mini-game. There were a lot of "on hindsight" logic actually required, like on reset: the timer needs to be stopped and reset, board array needs to be cleared before adding a new set, having to clear the winning message so it does not remain up during next game... which were all picked up upon testing different user decision permutations.

Is there anything in this code that you feel pleased about?
Pleased of the completion of all the sections and figuring out some minor bugs that were fixed, even though there was only a very slim chance it might surface.

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.

1 participant