Skip to content

Fix #9122#9176

Closed
alchzh wants to merge 1 commit intofacebook:masterfrom
alchzh:patch-1
Closed

Fix #9122#9176
alchzh wants to merge 1 commit intofacebook:masterfrom
alchzh:patch-1

Conversation

@alchzh
Copy link

@alchzh alchzh commented Mar 14, 2017

alternatively it could just tell you to the whole tag with

<button className="square" onClick={() => alert('click')}>
  {/* TODO */}
</button>

or something like that

@alchzh
Copy link
Author

alchzh commented Mar 14, 2017

Of note, the tutorial probably needs an overhaul

@gaearon
Copy link
Collaborator

gaearon commented Apr 21, 2017

Thanks! There are other PRs addressing this in more detail so going to use them instead.

@gaearon gaearon closed this Apr 21, 2017
@gaearon
Copy link
Collaborator

gaearon commented May 1, 2017

@brigand and I rewrote some parts of the tutorial in #9454, and following along should be easier now. Let us know if you're still struggling! We’re happy to look at remaining issues in case we missed anything.

https://facebook.github.io/react/tutorial/tutorial.html

@alchzh
Copy link
Author

alchzh commented May 1, 2017

Good work 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants