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 page 'Code-Splitting' #55

Merged
merged 9 commits into from
Mar 10, 2019
Merged

Translate page 'Code-Splitting' #55

merged 9 commits into from
Mar 10, 2019

Conversation

linsolas
Copy link
Contributor

Hello,

Voici ma traduction pour la page Code-Splitting, que j'ai traduit par "Fractionnement de code". Je suis ouvert à une meilleure traduction si vous avez.

@linsolas
Copy link
Contributor Author

Ah oui, un point aussi : j'ai traduit Error Boundaries par Limite d'erreur, mais je n'en suis pas trop content...
Comme je propose de passer à la traduction de la page dédiée à ce sujet, je veux bien connaître la meilleure traduction pour ce terme 😅

@Nab69 Nab69 added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 20, 2019
@Nab69
Copy link
Member

Nab69 commented Feb 20, 2019

Ah oui, un point aussi : j'ai traduit Error Boundaries par Limite d'erreur, mais je n'en suis pas trop content...
Comme je propose de passer à la traduction de la page dédiée à ce sujet, je veux bien connaître la meilleure traduction pour ce terme 😅

@linsolas Ouais j'avoue que "limite d'erreur" c'est pas un terme que j'utiliserais en conf ou en formation 😆
"limitation des erreurs" ou "Gestion des erreurs" (pour le titre par exemple) mais je dirais surtout que pour se faire mieux comprendre ca dépendra surtout du contexte de la phrase 😄

@linsolas
Copy link
Contributor Author

Hello @Nab69

En y réfléchissant, je me demande si Limitateur d'erreur ne serait pas plus adapté dans ce contexte.

C'est un concept introduit par React (v16) pour "attraper" les erreurs des composants enfants pour qu'elles évitent de se propager à toute l'application. Ce concept définit donc des "frontières" que ne sont pas censées dépasser les erreurs (offre soumise à conditions).

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.

de petits retours
avec juste une interrogation sur fractionnement, je suis plus habitue a lire separation

concernant limite d'erreur je trouve pas ca super jolie en france, et les derive son moyen. On ne pourrait pas garder error bounderies avec une NdT au debut ?

@@ -1,20 +1,16 @@
---
id: code-splitting
title: Code-Splitting
title: Fractionnement de code
Copy link
Collaborator

Choose a reason for hiding this comment

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

Fractionnement ne me gene pas, mais separation de code n'irait il pas aussi ?

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'ai changé pour "séparation", mais d'un point de vue personnel, aucune des deux traductions ne me satisfait vraiment...

if you are including large third-party libraries. You need to keep an eye on
the code you are including in your bundle so that you don't accidentally make
it so large that your app takes a long time to load.
Créer des bundles est génial, mais au fur et à mesure que votre application grandit, votre bundle va grandir aussi. En particulier si vous intégrez des bibliothèques tierces. Vous devez garder un œil sur le code que vous intégrez dans votre bundle pour éviter de le rendre si lourd que le chargement de votre application prendra beaucoup de temps.
Copy link
Collaborator

Choose a reason for hiding this comment

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

votre bundle va grandir aussi -> votre bundle va grossir aussi
ca permet d'eviter la repetition, mais facultatif comme changement

supported by bundlers like Webpack and Browserify (via
[factor-bundle](https://github.com/browserify/factor-bundle)) which can create
multiple bundles that can be dynamically loaded at runtime.
Pour éviter de vous retrouver avec un bundle trop volumineux, il est bon d'anticiper les problèmes et de commencer à fractionner votre bundle. Le [fractionnement de code](https://webpack.js.org/guides/code-splitting/) est une fonctionnalité supportée par les outils de génération de bundles tels que Webpack ou Browserify (*via* [factor-bundle](https://github.com/browserify/factor-bundle)) qui permet de créer plusieurs bundles pouvant être chargés dynamiquement au moment de l'exécution.
Copy link
Collaborator

Choose a reason for hiding this comment

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

une raison pour via au lieu de via comme en anglais ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Une erreur de ma part. La préposition via n'est effectivement pas tirée du latin, comme je le pensais. L'utilisation de l'italique n'a donc pas lieu d'être. Merci :)


Here's an example of how to setup route-based code splitting into your app using
libraries like [React Router](https://reacttraining.com/react-router/) with `React.lazy`.
Voici un exemple de configuration de fractionnement de code basé sur les routes à intégrer dans votre application en utilisant des bibliothèques telles que [React Router](https://reacttraining.com/react-router/) avec `React.lazy`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

les routes à intégrer -> les routes, à intégrer
petite virgule de separation je pense


`React.lazy` currently only supports default exports. If the module you want to import uses named exports, you can create an intermediate module that reexports it as the default. This ensures that treeshaking keeps working and that you don't pull in unused components.
`React.lazy` ne supporte actuellement que les exports par défaut. Si le module que vous souhaitez importer utilise des exports nommés, vous pouvez créer un module intermédiaire qui les réexportera par défaut. Cela garantit que le *treeshaking* ([procédé permettant de supprimer le code mort](https://developer.mozilla.org/fr/docs/Glossaire/Tree_shaking) — NdT) continue de fonctionner et que vous n'intégrez pas de composants inutilisés.
Copy link
Collaborator

Choose a reason for hiding this comment

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

il y a une faute en anglais aussi mais treeshaking est separe
donc tree shaking

@linsolas
Copy link
Contributor Author

Merci @sheplu pour ta relecture. Le texte a été mis à jour.

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for fr-reactjs ready!

Built with commit aa93367

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

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for fr-reactjs ready!

Built with commit 28d50e2

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

@linsolas linsolas mentioned this pull request Feb 25, 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 @linsolas !

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

Bon, comme d'hab', l'essentiel de mes retours sont des petits détails (par exemple, c’est pas « séparation du code » mais « découpage dynamique de code », cf. nav.yml, ce qui a un impact assez large…), dont certains se retrouvent de trad en trad vu que tu en as pondu plein avant que j'ai le temps de faire ma première revue 😉

Il a fallu éclaircir quelques points, mais au global c'est bien cool. Hâte que tu traites ça pour que je puisse finaliser l'intégration.

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 !

content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.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 Mar 8, 2019
@tdd tdd self-assigned this Mar 8, 2019
@tdd
Copy link
Collaborator

tdd commented Mar 9, 2019

@linsolas y'a 2 suggestions que t'as pas pris sans dire pkoi…?

@linsolas
Copy link
Contributor Author

linsolas commented Mar 9, 2019

Hello @tdd,

L’un est un oubli, l’autre est empêchée par Github qui me dit qu’aucun changement n’est présent !
5353B0F8-D4E9-4ECF-8E5F-C00D0F163214

@tdd
Copy link
Collaborator

tdd commented Mar 9, 2019

@linsolas j'ai corrigé ma suggestion.

@tdd tdd merged commit 028412f into reactjs:master Mar 10, 2019
@linsolas linsolas deleted the code-splitting branch March 10, 2019 08:52
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.

4 participants