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

Update conway-game-of-life.md #57

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

Conversation

boazblake
Copy link

I cleaned up the code

I cleaned up the code
@tbreuss
Copy link
Owner

tbreuss commented Nov 27, 2021

👍 A big clean up. Thanks!

This is looking good. I will later run the example in dev environment and merge your pull request.

@tbreuss tbreuss self-requested a review November 28, 2021 07:12
@tbreuss
Copy link
Owner

tbreuss commented Nov 28, 2021

I playing with GitHubs review features. In case you receive too much comments or so...

Copy link
Owner

@tbreuss tbreuss left a comment

Choose a reason for hiding this comment

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

Very well done.

One thing I want to mention: I was able to start the game, but I was not sure if I handled it correctly. So, a default setup would be nice, or a minimal instruction how to "play the game". I hope you understand what I mean.

const range = R.range
const without = R.without
const values = R.values
const prop = R.prop
const root = document.getElementById('GameOfLife')
Copy link
Owner

@tbreuss tbreuss Nov 28, 2021

Choose a reason for hiding this comment

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

The HTML element with ID #GameOfLife doesn't exist. So either change it to document.body or add a minimal HTML code snippet with the ID you selected. Example see here: https://github.com/tbreuss/mithril-by-examples/blob/main/examples/tweeter-box.md#markup

@boazblake
Copy link
Author

That's a good suggestion. I was wondering how intuitive it actually was to someone who was new to the concept - ill update it when I get some free time with a starting scene and some comments 👍🏼

@tbreuss
Copy link
Owner

tbreuss commented Dec 9, 2021

That's a good suggestion. I was wondering how intuitive it actually was to someone who was new to the concept - ill update it when I get some free time with a starting scene and some comments 👍🏼

Great! Thank you for your contribution. And keep coding in Mithril 😄

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