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

Finished User Interface Setup section in Part 1: Sudoku Board Setup #81

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

Conversation

mathonsunday
Copy link

In the post you have the line "We'll now define a class for our UI, SudokuUI(Frame), and go through each function." So I went through each function in the class.

Note: I will be working on more of the tutorial. I am just sending PRs as I finish logical chunks.

@merwok
Copy link
Contributor

merwok commented Jan 19, 2014

Thank you for doing this!

@econchick
Copy link
Owner

@mathonsunday - just wanted to let you know that I haven't forgotten about this! Hopefully I can set time to review next week. Thank you so much!


```python
def __draw_grid(self):
Copy link
Contributor

Choose a reason for hiding this comment

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

This first line is missing four spaces, like other code snippets in this file (the first and last one are good).

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