Skip to content
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

🌐 Translate "Rules of Hooks" into French #53

Merged
merged 15 commits into from Mar 1, 2019
Merged

Conversation

charlyx
Copy link
Contributor

@charlyx charlyx commented Feb 18, 2019

Bonjour 👋

Voici la traduction de "Rules of Hooks".

C'est ma toute première traduction officielle. J'ai donc préféré rester dans les "clous" contrairement à d'habitude.

Dans l'attente de vos reviews 😅

@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 19, 2019
Copy link
Collaborator

@sheplu sheplu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

juste une doute sur la traduction de top level
soit changer de terme, soit peut etre faire une NdT a la premiere occurence pour expliquer le terme, et rester sur top level apres ?


### Only Call Hooks at the Top Level {#only-call-hooks-at-the-top-level}
### Appelez les Hooks uniquement en premier lieu {#only-call-hooks-at-the-top-level}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

je suis pas specialement fan de premier lieu pour top level, mais d'un autre cote je n'ai pas mieux a proposer :/

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

peut etre prermier niveau

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moi aussi, je ne sais jamais comment le traduire en français 😕

ok, pour premier niveau 👌


**Don't call Hooks inside loops, conditions, or nested functions.** Instead, always use Hooks at the top level of your React function. By following this rule, you ensure that Hooks are called in the same order each time a component renders. That's what allows React to correctly preserve the state of Hooks between multiple `useState` and `useEffect` calls. (If you're curious, we'll explain this in depth [below](#explanation).)
**N'appelez pas de Hooks à l'intérieur de boucles, conditions ou de fonctions imbriquées.** À la place, utilisez seulement les Hooks en premier lieu dans votre fonction React. En suivant cette règle, vous assurez que les Hooks sont appelés dans le même ordre à chaque fois que le composant est rendu. C'est ce qui permet à React de garantir le bon état des Hooks entre plusieurs appels de `useState` et `useEffect`. (Si vous êtes curieux, nous expliquerons cela en détails [ci-dessous](#explanation).)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

premier lieu -> premier niveau ?

@netlify
Copy link

netlify bot commented Feb 23, 2019

Deploy preview for fr-reactjs ready!

Built with commit c89f894

https://deploy-preview-53--fr-reactjs.netlify.com

Copy link
Collaborator

@tdd tdd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salut @charlyx !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main. C'est globalement très bien, on trouve juste quelques couacs récurrents :

  • « assurer » au lieu de « garantir », qui passe en général mieux (mais pas tout le temps, j'en ai laissé un)
  • « à la place », expressément interdit par le Glossaire
  • « Dans le futur », à proscrire au profit de « À l’avenir » (ibid.)
  • « Conserver » (trop générique) au lieu de « persister »
  • Le terme « rendu », parfois à garder mais trop souvent remplaçable avantageusement par la simplification « affichage » (cf. MP / Glossaire)

Bref, des détails, rien de vraiment majeur 😉

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 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.

Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans master !

@@ -1,38 +1,38 @@
---
id: hooks-rules
title: Rules of Hooks
title: Les règles des Hooks
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Essaie d'ajouter la traduction identique dans content/docs/nav.yml à ta PR (sinon je le ferai après fusion).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'avais hésité à le faire. Je le fais de ce pas 😄

content/docs/hooks-rules.md Outdated Show resolved Hide resolved
content/docs/hooks-rules.md Outdated Show resolved Hide resolved
content/docs/hooks-rules.md Outdated Show resolved Hide resolved
content/docs/hooks-rules.md Outdated Show resolved Hide resolved
content/docs/hooks-rules.md Outdated Show resolved Hide resolved
content/docs/hooks-rules.md Outdated Show resolved Hide resolved
content/docs/hooks-rules.md Outdated Show resolved Hide resolved
content/docs/hooks-rules.md Outdated Show resolved Hide resolved
content/docs/hooks-rules.md Outdated Show resolved Hide resolved
@tdd tdd added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Feb 28, 2019
tdd and others added 5 commits February 28, 2019 22:20
Co-Authored-By: charlyx <charleshenri.guerin@me.com>
Co-Authored-By: charlyx <charleshenri.guerin@me.com>
Co-Authored-By: charlyx <charleshenri.guerin@me.com>
Co-Authored-By: charlyx <charleshenri.guerin@me.com>
Co-Authored-By: charlyx <charleshenri.guerin@me.com>
@tdd
Copy link
Collaborator

tdd commented Feb 28, 2019

@charlyx par pitié pas de commits individuels par suggestion, ça pourrit ma mailbox et l'histo ! La review de PR demande explicitement d'utiliser le groupement par lots !

tdd and others added 2 commits February 28, 2019 22:24
Co-Authored-By: charlyx <charleshenri.guerin@me.com>
Co-Authored-By: charlyx <charleshenri.guerin@me.com>
@charlyx
Copy link
Contributor Author

charlyx commented Feb 28, 2019

@tdd Oops, mille excuses.

tdd and others added 3 commits February 28, 2019 22:33
@tdd tdd merged commit d374042 into reactjs:master Mar 1, 2019
@charlyx charlyx deleted the hooks-rules branch March 1, 2019 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants