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

Traduction française de l'article de blog “Introducing the React profiler“ #248

Merged
merged 1 commit into from
Feb 13, 2020

Conversation

tdd
Copy link
Collaborator

@tdd tdd commented Feb 12, 2020

Salut @JeremiePat et la team !

Si vous avez un moment sous 24h (je sais, je sais…) j'apprécierais votre relecture de ma trad pour l’article de blog sur le Profileur. /ping aux grand·e·s habitué·e·s : @juliettelofaro @lbelavoir @linsolas @forresst @LaureRC

Je suis en train de traduire les articles de blog listés dans l'issue centrale (liés 2+ fois depuis les docs) ainsi que certains articles autres mais ayant moins de 2 ans.

Merci !

@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 12, 2020
@tdd tdd self-assigned this Feb 12, 2020
@netlify
Copy link

netlify bot commented Feb 12, 2020

Deploy preview for fr-reactjs ready!

Built with commit 7a38764

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

> A production profiling bundle is also available as `react-dom/profiling`.
> Read more about how to use this bundle at [fb.me/react-profiling](https://fb.me/react-profiling)
> `react-dom` 16.5+ prend en charge le profilage en mode DEV.
> Un bundle de production apte au profilage est également disponible : `react-dom/profiling`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Question rapide, on ne traduit pas bundle ? Je n'ai pas trouvé dans le glossaire ou les bonnes pratiques.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Non, on le traduit pas (il est présent dans plusieurs autres pages) faute d'une trad clean. “Paquet” est plus généralement employé pour les modules npm, pas pour les bundles applicatifs produits par un bundler type Webpack. Donc on garde tel quel…


* The **render** phase determines what changes need to be made to e.g. the DOM. During this phase, React calls `render` and then compares the result to the previous render.
* The **commit** phase is when React applies any changes. (In the case of React DOM, this is when React inserts, updates, and removes DOM nodes.) React also calls lifecycles like `componentDidMount` and `componentDidUpdate` during this phase.
* La phase de **rendu** détermine quelles modifications doivent être effectuées, par exemple au DOM. Durant cette phase, React appelle `render` et compare son résultat au rendu précédent.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* La phase de **rendu** détermine quelles modifications doivent être effectuées, par exemple au DOM. Durant cette phase, React appelle `render` et compare son résultat au rendu précédent.
* La phase de **rendu** détermine quelles modifications doivent être effectuées, par exemple au DOM. Durant cette phase, React appelle `render` et compare son résultat au rendu précédent.

Il y a quelques espaces en trop ça et là.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oui, vieille habitude, même si j'essaie d'arrêter. Sans importance : le HTML lissera.

@tdd tdd added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Feb 13, 2020
@tdd tdd merged commit e98449c into master Feb 13, 2020
@tdd tdd deleted the l10n/blog-react-profiler branch February 13, 2020 13: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.

2 participants