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

Guinevere's tic-tac-toe #22

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

Conversation

guineveresaenger
Copy link

Here it is. It was a struggle.
I couldn't figure out how to make my board squares hold their shape without assigning them heights in pixels, which of course broke the Foundation. Ems would shrink with window size, but also introduce extra space between the board columns. I never utilized the possibility of searching for a ready-made tic tac toe board. I think I spent over 4 hours on that one challenge alone.

I would still like to figure out how to delete my model from the database. I couldn't make the delete function work, and I ran into scope issues for 'this' while displaying game data. I used pure jQuery to append the information to my game history element. I even made a delete button with just jQuery, but then I ran into the issue of needing to find the particular game I wanted to delete by id. Perhaps add the particular game's ID to the html p tag somehow, and look it up by that? I would like to look up others' solutions to find out more!

I'm pretty happy with my game logic and tests. I also like that my game history goes away when New Game is clicked, and it updates automatically. Due to my struggles, my design suffered pretty badly. I know I can do better, given more time. Merry Christmas!

guineveresaenger and others added 30 commits December 13, 2016 12:28
…exports. Created basic Board attributes using BDD.
…ailable and to set a space with the given marker
…null board fields be a blank space string instead
…e logic lives. Tried and tried and tried to pass the current mark into the tmplate, and need to figure out rendering again.
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