-
Notifications
You must be signed in to change notification settings - Fork 163
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
Traduction of the "Managing state" page #458
Conversation
Coucou @KohdeAvekLeQ ! Merci pour ton taf ! Je relis ça ASAP. |
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 @KohdeAvekLeQ !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
Il y a bien sûr plein de petits trucs, mais au global je suis impressionné par la qualité de formulation sur plusieurs contenus complexes, tech ou non. Bravo !
Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)
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 (pas de commit direct !), et sur la fin de créer le commit groupé. (ce mode de regroupement en un seul commit est à favoriser absolument, mais tu devras aller dans l'onglet Files changed pour y accéder, il n'est pas utilisable dans l’onglet Conversation)
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. J'y recalerai si besoin les usual suspects : espaces insécables avant les ponctuations doubles, et apostrophes françaises.
Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans main
!
src/content/learn/managing-state.md
Outdated
|
||
Head over to [Reacting to Input with State](/learn/reacting-to-input-with-state) to start reading this chapter page by page! | ||
Rendez vous sur [Réagir à une entrée avec un état](/learn/reacting-to-input-with-state) pour commencer à lire ce chapitre page par page ! |
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.
Rendez vous sur [Réagir à une entrée avec un état](/learn/reacting-to-input-with-state) pour commencer à lire ce chapitre page par page ! | |
Allez sur [Réagir à la saisie avec un état](/learn/reacting-to-input-with-state) pour commencer à lire ce chapitre page par page ! |
src/content/learn/managing-state.md
Outdated
|
||
Or, if you're already familiar with these topics, why not read about [Escape Hatches](/learn/escape-hatches)? | ||
Ou, si vous êtes déjà familier avec ces sujets, pourquoi ne pas lire à propos des [Trappes d'évacuation](/learn/escape-hatches)? |
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.
Ou, si vous êtes déjà familier avec ces sujets, pourquoi ne pas lire à propos des [Trappes d'évacuation](/learn/escape-hatches)? | |
Ou alors, si vous êtes déjà à l’aise avec ces sujets, pourquoi ne pas explorer les [Échappatoires](/learn/escape-hatches)? |
Note que le terme VO m'embête à la base, j'en parle avec Dan. Mettre tout ce qui touche aux effets et hooks personnalisés dans "Escape hatches", même si je comprends l'intention, me semble problématique en termes de discours.
La trad entérinée est en tout cas "échappatoires".
src/sidebarLearn.json
Outdated
@@ -121,24 +121,24 @@ | |||
] | |||
}, | |||
{ | |||
"title": "Managing State", | |||
"title": "Gérer les états", |
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.
"title": "Gérer les états", | |
"title": "Gérer l'état", |
src/sidebarLearn.json
Outdated
"path": "/learn/managing-state", | ||
"tags": ["intermediate"], | ||
"routes": [ | ||
{ | ||
"title": "Reacting to Input with State", | ||
"title": "Réagir à une entrée avec un état", |
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.
"title": "Réagir à une entrée avec un état", | |
"title": "Réagir à la saisie avec un état", |
src/sidebarLearn.json
Outdated
"path": "/learn/choosing-the-state-structure" | ||
}, | ||
{ | ||
"title": "Sharing State Between Components", | ||
"title": "Partager l'état entre les composants", |
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.
"title": "Partager l'état entre les composants", | |
"title": "Partager l'état entre des composants", |
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.
Quitte à déjà traduire les noms de pages pour être cohérent avec tes propres liens, va jusqu'au bout 😄
Merci pour cette review, effectivement il y avait pas mal de petites erreurs, c'était très intéressant à lire ! |
Co-authored-by: Christophe Porteneuve <[email protected]>
Size changes📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 Three Pages Changed SizeThe following pages changed size from the code in this PR compared to its base branch:
DetailsOnly the gzipped size is provided here based on an expert tip. First Load is the size of the global bundle plus the bundle for the individual page. If a user were to show up to your website and land on a given page, the first load size represents the amount of javascript that user would need to download. If Any third party scripts you have added directly to your app using the Next to the size is how much the size has increased or decreased compared with the base branch of this PR. If this percentage has increased by 10% or more, there will be a red status indicator applied, indicating that special attention should be given to this. |
@KohdeAvekLeQ je t'ai attribué la première page demandée (réagir à la saisie…). Bonne trad' à toi ! |
Voilà ma PR pour la page Managing State, n'hésitez pas si vous avez des retours à faire dessus