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

Add lifecycle events documentation w/ working example 📚 #40

Merged
merged 4 commits into from
Feb 6, 2017

Conversation

danigb
Copy link
Contributor

@danigb danigb commented Feb 6, 2017

Add some documentation to README.md with a canvas example.

@jorgebucaran
Copy link
Owner

jorgebucaran commented Feb 6, 2017

@danigb Very nice! Thanks, what's ...?

EDIT: I see, can you provide a full working example? I can create a codepen for it and link to it later.

@danigb
Copy link
Contributor Author

danigb commented Feb 6, 2017

Great! The ... means insert your code here, but maybe I should remove it, isn't it? 😂

@jorgebucaran
Copy link
Owner

@danigb LOL, right. Can we have simple working example?

@danigb
Copy link
Contributor Author

danigb commented Feb 6, 2017

With a working example now

README.md Outdated

```js
function repaint(canvas, model) {
var ctx = canvas.getContext('2d')
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please spell this out, context i know its a pain, but to most people ctx means nothing! Love the contrib though! ❤️

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I didn't read you comment. Fixing it...

@jorgebucaran
Copy link
Owner

@danigb Awesome, thanks!

@jorgebucaran jorgebucaran merged commit e79eac2 into jorgebucaran:master Feb 6, 2017
@jorgebucaran jorgebucaran changed the title Add lifecycle events documentation. Related (and hopefully fixes) #12 Add lifecycle events documentation w/ working example 📚 Feb 6, 2017
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.

None yet

3 participants