-
Notifications
You must be signed in to change notification settings - Fork 166
conditionnal rendering #40
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques corrections pour toi @sheplu , j'en ai surement laissé passer, je referais une lecture ;)
|
||
Conditional rendering in React works the same way conditions work in JavaScript. Use JavaScript operators like [`if`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/if...else) or the [conditional operator](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Operators/Conditional_Operator) to create elements representing the current state, and let React update the UI to match them. | ||
Le rendu conditionel en React fonctionne de la même façon que les conditions en Javascript. On utilise l'opérateur Javascript [`if`](https://developer.mozilla.org/fr-FR/docs/Web/JavaScript/Reference/Statements/if...else) ou le [opérateur conditionnel](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Operators/Conditional_Operator) pour créer des éléments représentant l'état courrant, et on laisse à React le rôle de mettre à jour l'UI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je rajouterais bien "ternaire" comme dans MDN pour [opérateur conditionnel] qui deviendrais [opérateur (ternaire) conditionnel]. Je trouve ça un peu plus précis mais c'est vrai que ca n'est pas indiqué dans la doc en 'en'.
@@ -24,7 +24,7 @@ function GuestGreeting(props) { | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu as oublié de traduire les littéraux :)
Welcome back! -> Bienvenue!
Please sign up -> Veuillez vous inscrire.
|
||
You can use variables to store elements. This can help you conditionally render a part of the component while the rest of the output doesn't change. | ||
Vous pouvez utiliser des varibles pour stocker des éléments. Cela peut vous aider à rendre de façon conditonnelle une partie du composant pendant que le reste ne change pas. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
varibles -> variables
|
||
Consider these two new components representing Logout and Login buttons: | ||
Considérons ces deux nouveaux composants représentant les boutons de Logout et Login : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Est-ce qu'on les considererait pas aussi comme des littéraux Se connecter/Se déconnecter ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je me dis que Login/Logout sont des termes qui ne sont pas forcement traduits sur les sites en francais
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est vrai que ca depend des sites... mais je serais plus partisan de faire la trad sur ce coup
|
||
It will render either `<LoginButton />` or `<LogoutButton />` depending on its current state. It will also render a `<Greeting />` from the previous example: | ||
Dans l'exemple ci-dessous, nous allons créer un [composant stateful](/docs/state-and-lifecycle.html#adding-local-state-to-a-class) appelé `LoginControl`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
composant stateful -> composant à état
|
||
It works because in JavaScript, `true && expression` always evaluates to `expression`, and `false && expression` always evaluates to `false`. | ||
Cela fonctionne parce qu'en JavaScript, `true && expression` est toujours évalué à `expression`, et `false && expression` est toujours évalué à `false`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cela -> Ça
|
||
In the example below, we use it to conditionally render a small block of text. | ||
Dans l'exemple ci-dessous, on l'utilise pour rendre contionnellement un bloc de texte. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
contionnellement -> conditionnellement
(le séjour à Hawaï te réussit pas ^^)
@@ -186,13 +187,13 @@ render() { | |||
} | |||
``` | |||
|
|||
Just like in JavaScript, it is up to you to choose an appropriate style based on what you and your team consider more readable. Also remember that whenever conditions become too complex, it might be a good time to [extract a component](/docs/components-and-props.html#extracting-components). | |||
Tout comme en Javascript, c'est à vous de choisir un style approprié suivant ce que vous et votre équipe considérés plus facilement lisable. Rappelez vous aussi que lorsque les conditions deviennent trop complexes, il peut être le bon moment pour penser à [extraire un composant](/docs/components-and-props.html#extracting-components). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
considérés -> considerez
lisable -> lisible (voir même compréhensible pourquoi pas)
je partirais presque aussi sur une trad du genre :
"suivant ce qui semble le plus lisible pour vous et votre équipe"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
il peut être -> c'est peut être
|
||
### Preventing Component from Rendering {#preventing-component-from-rendering} | ||
### Empécher un Composant de se rendre {#preventing-component-from-rendering} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai un peu du mal avec le titre, que penses-tu de ça ?
"Empêcher qu'un Composant soit rendu"
|
||
In rare cases you might want a component to hide itself even though it was rendered by another component. To do this return `null` instead of its render output. | ||
Dans de rare cas, vous voulez qu'un composant soit capable de se masquer même si il est rendu par un autre composant. Pour ce faire, il suffit de retourner `null` à la place de son rendu. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
si il -> s'il ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Salut @sheplu !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
C'est ta première PR traitée, du coup on y trouve évidemment nombre de usual suspects. Ne te laisse pas démonter, c'est parfaitement normal, et ça ne déprécie en rien ton travail.
Tout est fait sous forme de Suggestions GitHub, il te suffit à chaque suggestion que tu approuves de cliquer le bouton d'ajout au lot de modifs à committer, et sur la fin de créer le commit groupé (ce bouton est dispo dans la vue Files changed, mais pas la vue Conversation de la PR).
Si certaines appellent discussion, n'hésite pas, tu réponds, on en parle, on avance ensemble 🤝
Quand tout sera bouclé côté code, je ferai une repasse visuelle finale en local avant de fusionner.
Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans master
!
Co-Authored-By: sheplu <sheplu@users.noreply.github.com>
Deploy preview for fr-reactjs ready! Built with commit 5d97aea |
Deploy preview for fr-reactjs ready! Built with commit b0c0c60 |
This includes translating literals in code samples
Merci pour la review @tdd |
@sheplu et oui c'est tout l'intérêt d'attendre ses premières reviews pour ses PRs suivantes, même si ça fait un peu ronger son frein… 😉 . Dans tes PRs à venir, pense à mettre à jour le libellé de la page (si nécessaire) dans |
Conditionnal rendering