Skip to content

Update tutorial.md #366

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

Merged
merged 7 commits into from
Feb 20, 2017
Merged

Update tutorial.md #366

merged 7 commits into from
Feb 20, 2017

Conversation

danpadmore
Copy link

Correct the highlighted line to match the change that is being discussed, namely that the CommentForm can call the callback when the user submits the form

Dan Padmore added 2 commits January 7, 2017 12:46
Correct the highlighted line to match the change that is being discussed, namely that the CommentForm can call the callback when the user submits the form
Also highlight changed line where onSubmit event handler is added to form tag.
@Daniel15
Copy link
Member

Daniel15 commented Jan 7, 2017 via email

@danpadmore
Copy link
Author

You're right, the ASP.NET 4 tutorial needs to be updated in the same way.

I'll attempt to include it in this pull request.

@Daniel15
Copy link
Member

Daniel15 commented Jan 7, 2017 via email

Dan Padmore added 5 commits January 7, 2017 13:12
Corrects the highlighted line in aspnet4 tutorial, to match the change that is being discussed, namely that the CommentForm can call the callback when the user submits the form
Also highlight changed line in aspnet4 tutorial, where onSubmit event handler is added to form tag.
Highlight complete block of added code lines.
Highlight complete block of added codes lines, in optimization part of tutorial.
Fix highlighted code lines in tutorial
@danpadmore
Copy link
Author

There it is :)

I didn't expect to do more than 1 change, so I had some more changes on a separate branch in my fork.
I merged them also into the branch for this pull request.
All of the changes are related to highlighting code lines in the tutorials (core/aspnet4)

@Daniel15 Daniel15 merged commit 5892e51 into reactjs:master Feb 20, 2017
@Daniel15
Copy link
Member

Thanks!

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