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

My tic-tac-toe! #11

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

My tic-tac-toe! #11

wants to merge 26 commits into from

Conversation

digivava
Copy link

@digivava digivava commented Jul 1, 2016

I had soooo much fun with this project! I did a lot of whiteboarding before writing any code, and worked hard to implement exactly the design I had in my head.
I also had fun thinking about ways to make the UX better (still a long way to go). And I found myself "thinking in callbacks" for the first time, which was exciting.

There's one major bug, which is that the turn doesn't switch after you choose a space that's already been claimed. (It still notifies you that you chose an already-taken space though.) I think I might have to significantly change the ways some of my functions work to fix that.

digivava added 25 commits June 30, 2016 10:55
…ust for my own visibility, will change to something prettier later. Also gave data-position to each cell of table for referring to later.
…uldn't initialize, and made it so click handler connects to take_space method properly
…e to cat's game because that's what it's called in tic-tac-toe.
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