Translation of the “Mixins Considered Harmful“ blog post #133
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
tdd
added
the
Pending Review
|
Deploy preview for fr-reactjs ready! Built with commit d3bd5a9 |
JeremiePat
requested changes
Mar 29, 2019
|
|
||
| Another group of mixins we discovered were helpers for providing and consuming [React context](/docs/context.html). Context is an experimental unstable feature, has [certain issues](https://github.com/facebook/react/issues/2517), and will likely change its API in the future. We don’t recommend using it unless you’re confident there is no other way of solving your problem. | ||
| Nous avons aussi remarqué une autre catégorie de mixins, qui étaient des utilitaires pour interagir avec le [Contexte React](/docs/context.html). Le Contexte est une fonctionnalité expérimentale instable qui a [quelques soucis](https://github.com/facebook/react/issues/2517), et nous changerons sans doute son API à l’avenir *(ce qu’ils ont fait avec React 16.3, NdT)*. Nous déconseillons de l’utiliser à moins que vous soyez certain·e qu’il représente la seule solution à votre problème. |
There was a problem hiding this comment.
Ce serait bien d'avoir un lien dans ta note. au choix vers https://fr.reactjs.org/docs/context.html ou vers https://fr.reactjs.org/blog/2018/03/29/react-v-16-3.html#official-context-api (le deuxième est sans doute plus pertinent dans le contexte, c'est la suggestion que je fait)
There was a problem hiding this comment.
Ben en même temps y'a un lien vers la nouvelle API dans le texte principal de la remarque, du coup j'ai évité.
JeremiePat
added
Reviewed
and removed
Pending Review
Co-Authored-By: tdd <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Salut @JeremiePat, je review ta propre trad aujourd'hui ! Voici un autre pavé pour le plaisir de tes yeux… 😜