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 Higher-Order Components #64

Merged
merged 32 commits into from
Mar 14, 2019
Merged

Translate Higher-Order Components #64

merged 32 commits into from
Mar 14, 2019

Conversation

robinmetral
Copy link
Contributor

@robinmetral robinmetral commented Feb 24, 2019

PR pour la traduction de la page Higher-Order Components.

Progression

Lundi 11 mars: 399/399 lignes (100%) 🎉

  • Use HOCs For Cross-Cutting Concerns
  • Don’t Mutate the Original Component. Use Composition.
  • Convention: Pass Unrelated Props Through to the Wrapped Component
  • Convention: Maximizing Composability
  • Convention: Wrap the Display Name for Easy Debugging
  • Caveats
    • Don’t Use HOCs Inside the render Method
    • Static Methods Must Be Copied Over
    • Refs Aren’t Passed Through

Difficultés avec certains termes techniques

  • "wrapped component" ➡️ souvent traduit "composant enfant"

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for fr-reactjs ready!

Built with commit 668c636

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

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for fr-reactjs ready!

Built with commit fd4f17d

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

@robinmetral
Copy link
Contributor Author

@tdd je crois que j'ai été un peu optimiste en disant que ça allait être prêt d'ici ce soir... vu mon rythme, j'aurai probablement besoin de quelques jours. J'espère que ça ne pose pas de problèmes !

@JeremiePat
Copy link
Collaborator

T'inquiète, on a plein de PR à review et à merge donc pas de problème si tu as besoins de quelques jours ;)

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

Salut @robinmetral Tu t'en sort ? Tu pense pouvoir finir cette traduction quand ?

@tdd tdd self-assigned this Mar 8, 2019
@robinmetral
Copy link
Contributor Author

Hello @JeremiePat, j'ai été débordé et la trad est passée à la trappe ces derniers jours. Je termine la PR ce weekend!

@robinmetral
Copy link
Contributor Author

Terminé @JeremiePat @tdd 🎉

@JeremiePat JeremiePat self-requested a review March 11, 2019 20:04
@robinmetral robinmetral marked this pull request as ready for review March 12, 2019 07:04
@tdd tdd added Reviewing Un mainteneur qualifié est en train de composer la revue de code and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Mar 13, 2019
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 @robinmetral !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.

Écoute franchement, c'est pas mal du tout 👏 — surtout pour une première trad ! Beaucoup de bonnes formules, de bonnes initiatives de phrasé, etc.

L'essentiel des retour se résume à quelques points ultra-récurrents :

  • « composant enrobé » au lieu de « composant enfant » (pas tout à fait le même sens, surtout si on compose les HOC)
  • « enrober » plutôt que « envelopper » (les deux se valent mais on a normalisé sur le premier dans toutes les PR déjà validées)
  • Espaces insécables qui manquaient, et littérales quand elles sont déjà là
  • Quelques trads insistant à tort sur le comment au lieu du *pourquoi (commentaires de code)
  • Faux-amis classiques (ex. « souscription »)
  • Lire/apprendre « davantage » au lieu du moche « plus » (cf. MP / Glossaire)
  • « Modifier », simple et concis, au lieu de l'ampoulé « faire une mutation »

Quelques trucs plus précis de loin en loin, mais globalement c'est super cool, bravo 👏

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.

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

content/docs/nav.yml Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
@tdd tdd added Reviewed and removed Reviewing Un mainteneur qualifié est en train de composer la revue de code labels Mar 14, 2019
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
@tdd
Copy link
Collaborator

tdd commented Mar 14, 2019

@robinmetral prends bien soin de ne pas faire un commit par suggestion, par pitié ! C'est dit en large et en travers dans mon intro de revue ! Ça pourrirait mes notifs et l'historique.

tdd and others added 8 commits March 14, 2019 15:08
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
@robinmetral
Copy link
Contributor Author

robinmetral commented Mar 14, 2019

@robinmetral prends bien soin de ne pas faire un commit par suggestion, par pitié ! C'est dit en large et en travers dans mon intro de revue ! Ça pourrirait mes notifs et l'historique.

@tdd désolé, j'avais pas vu cette section - je vais rollback et reprendre

@tdd
Copy link
Collaborator

tdd commented Mar 14, 2019

@robinmetral je vais finir par le mettre en gras énorme au début, j'ai un traducteur sur 4 qui me fait le coup ! 😢 . Fais pas de rollback, juste batche la suite 😉

tdd and others added 2 commits March 14, 2019 15:37
Co-Authored-By: robinmetral <35560568+robinmetral@users.noreply.github.com>
@robinmetral
Copy link
Contributor Author

Je crois que c'est tout bon de mon côté @tdd

Franchement, un grand merci pour l'effort que tu mets dans les reviews, j'ai appris plein de choses ! Je me réjouis de ma prochaine traduction 🙂 (et promis, je batcherai les commits)

@tdd tdd merged commit 8395454 into reactjs:master Mar 14, 2019
@robinmetral robinmetral deleted the higher-order-components branch March 15, 2019 09:50
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.

3 participants