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

tic tac toes #15

Open
wants to merge 1 commit into
base: grangerdanger
Choose a base branch
from
Open

tic tac toes #15

wants to merge 1 commit into from

Conversation

lgranger
Copy link

Things I would like to do:

  • Add css and make it prety
  • Not allow further clicking on divs once a game has been won
  • Ability to switch from X as the starting marker to O as the starting marker

@Brennaleker Brennaleker self-assigned this Feb 9, 2016
@@ -1,3 +1,25 @@
html {

Choose a reason for hiding this comment

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

lines 1-3 are unused, you could remove those

@Brennaleker
Copy link

Well done, most of my comments are stylistic and nitpicky. That is always a good sign, it means that your logic was sound and code was clear. Well done and welcome to Javascript! 🐙 🌟

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