This repository has been archived by the owner on Apr 21, 2023. It is now read-only.
Sync with reactjs.org @ 80c16779 #78
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Link each mention of a React API in rendering-elements.md to the reference docs. On this page replace every static occurrence of `ReactDOM.render()` by link to the reference docs [`ReactDOM.render()`](/docs/react-dom.html#render) Is this in the right direction @sophiebits ? 🤔 ref #2630
…Mode (#2810) This landed in 16.13 via facebook/react#17942
`<renderer>.update()` doesn't return anything
Think of "myriad" as a number. You wouldn't say "a nine of things", you would say "nine things".
1- Added getDerivedStateFromError lifecycle method to "Do Hooks cover all use cases for classes?" 2- Added getSnapshotBeforeUpdate lifecycle method to "How do lifecycle methods correspond to Hooks?"
* added stackblitz to playground * modified link of Stackblitz I changed the link based on comment from @EricSimons
This reverts commit 869c740.
adding next section URL path at the end in order to ease readers follow steps
adding the next section path at the end.
Clarifying a useState sentence
* Fix typo in hooks-faq.md * Update hooks-faq.md * Update hooks-faq.md Co-authored-by: Sophie Alpert <[email protected]> Co-authored-by: Sunil Pai <[email protected]>
- correcting assertion mistakes
`MySlowList` "lag behind" => `MySlowList` "lags behind"
* Update components docs for functional components * Update components-and-props.md Co-authored-by: Sunil Pai <[email protected]>
"You can use in new code" => "You can use them in new code"
Missing code
According to [ProofreadNOW](https://www.proofreadnow.com/blog/bid/101485/If-I-Were-or-If-I-Was-Which-is-Correct), > Guideline: Use were (instead of was) in statements that are contrary to fact.
Co-authored-by: Juan Manuel Rinaudo <[email protected]> Co-authored-by: Sunil Pai <[email protected]>
I believe it was meant to be "news story", but I could be misunderstanding something in the context of the paragraph.
* Update legacy lifecycle methods componentWillReceiveProps -> componentDidUpdate * Update legacy lifecycle methods componentWillReceiveProps -> componentDidUpdate * Update higher-order-components.md Co-authored-by: Sunil Pai <[email protected]>
* Update README.md (improve usability) Clear up the ambiguous "Running locally" to the anchor link for better usability * Update README.md Co-authored-by: Sunil Pai <[email protected]>
…"Next article" links => Issue Resolved (#3082)
Added and updated copyright headers. Added some missing Flow types. Removed an invalid prop-types import.
* Added padding to the left and to the right * Switched it to MetaTitle
Co-authored-by: Nikhil Sharma <[email protected]>
* Correct grammar in conditional-rendering.md * Correct grammar in content/docs/conditional-rendering.md
* Update urls of tools examples Some urls are outdated, they still redirect to the correct page but for example the info you can see on the bottom left of the page to know which page you're about to visit when hovering the link is wrong. - https://github.com/facebookincubator/create-react-app -> https://github.com/facebook/create-react-app - https://github.com/zeit/next.js/ -> https://github.com/vercel/next.js * Provide urls to tools websites instead of GH repos
…3136) * Removed deadlink from Event System section and added correct links * Changed wording of Event System section and removed react-native events link * Update codebase-overview.md Co-authored-by: Dowen Robinson <[email protected]> Co-authored-by: Dan Abramov <[email protected]>
* Update hooks-overview.md Changed 'is' -> 'are' * Update content/docs/hooks-overview.md Co-authored-by: Ricky <[email protected]> Co-authored-by: Ricky <[email protected]>
This page is focused on vanilla React examples.
|
Deploy preview for ta-reactjs ready! Built with commit 5079263 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR was automatically generated.
Merge changes from reactjs.org at 80c1677
The following files have conflicts and may need new translations:
Please fix the conflicts by pushing new commits to this pull request, either by editing the files directly on GitHub or by checking out this branch.
DO NOT SQUASH MERGE THIS PULL REQUEST!
Doing so will "erase" the commits from master and cause them to show up as conflicts the next time we merge.