-
Notifications
You must be signed in to change notification settings - Fork 8
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
base: main
Are you sure you want to change the base?
Conversation
I cleaned up the code
👍 A big clean up. Thanks! This is looking good. I will later run the example in dev environment and merge your pull request. |
I playing with GitHubs review features. In case you receive too much comments or so... |
There was a problem hiding this 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') |
There was a problem hiding this comment.
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
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 😄 |
I cleaned up the code