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

Premiere traduction du Strict mode #73

Closed
wants to merge 1 commit into from
Closed

Premiere traduction du Strict mode #73

wants to merge 1 commit into from

Conversation

erkela
Copy link
Contributor

@erkela erkela commented Feb 28, 2019

Salut la team,

Voici ma proposition pour la traduction de la page Strict mode.
J'attend vos retour. 😊

@netlify
Copy link

netlify bot commented Feb 28, 2019

Deploy preview for fr-reactjs ready!

Built with commit 319cef6

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

@Nab69 Nab69 added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Mar 1, 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.

certaines erreurs que j'ai vu, pas mal d'accents en général
normalement tu as juste a verifier mes changements et juste les accepter si ca te semble valide

content/docs/strict-mode.md Show resolved Hide resolved
content/docs/strict-mode.md Show resolved Hide resolved

Additional functionality will be added with future releases of React.
D'autres fonctionnalités seront ajoutées dans une future version de React.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
D'autres fonctionnalités seront ajoutées dans une future version de React.
D'autres fonctionnalités seront ajoutées dans les futures versions de React.

content/docs/strict-mode.md Show resolved Hide resolved
content/docs/strict-mode.md Show resolved Hide resolved

For example, consider the following code:
Par exemple, examinez le code suivant:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Par exemple, examinez le code suivant:
Par exemple, examinez le code suivant :

content/docs/strict-mode.md Show resolved Hide resolved
content/docs/strict-mode.md Show resolved Hide resolved
content/docs/strict-mode.md Show resolved Hide resolved
content/docs/strict-mode.md Show resolved Hide resolved
@tdd tdd self-assigned this Mar 8, 2019
@tdd
Copy link
Collaborator

tdd commented Mar 13, 2019

@erkela outre ma review qui devrait arriver ce jour, comment se fait-il que tu n'aies rien traité de la review de @sheplu y'a 11 déjà jours ?

@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 14, 2019
@tdd tdd self-requested a review March 14, 2019 14:36
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 @erkela !

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

Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)

Bon, ça démarre douloureusement, puis tu trouves ton rythme et les tournures s'améliorent. Les trois principales sources de retour semblent être :

  • Tu es super fâché avec les accents graves 😁
  • Tu es super fâché avec les espaces (insécables qui plus est) avant les signes de ponctuation doubles (cf. Meilleures Pratiques de Traduction dans le wiki)
  • Il semble que tu ne connaisses pas assez bien pas mal de points abordés, du coup y'a plusieurs contre-sens flagrants.

Ce sont des choses qui arrivent ! Et c'est ta première trad', alors ne sois pas trop dur avec toi-même, c'est déjà super cool d'avoir fourni tout ce boulot !

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 !

permalink: docs/strict-mode.html
---

`StrictMode` is a tool for highlighting potential problems in an application. Like `Fragment`, `StrictMode` does not render any visible UI. It activates additional checks and warnings for its descendants.
`StrictMode` est un outil pour signaler les problémes potentiels qui peuvent apparaître dans une application. Tout comme `Fragment`, `StrictMode` n'effectue aucun rendu visuel. Il active des vérifications et avertissement supplémentaire pour ses descendants.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
`StrictMode` est un outil pour signaler les problémes potentiels qui peuvent apparaître dans une application. Tout comme `Fragment`, `StrictMode` n'effectue aucun rendu visuel. Il active des vérifications et avertissement supplémentaire pour ses descendants.
`StrictMode` est un outil pour détecter les problèmes potentiels d’une application. Tout comme `Fragment`, `StrictMode` n'affiche rien. Il active des vérifications et avertissements supplémentaires pour ses descendants.


> Note:
> Remarque:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
> Remarque:
> Remarque

Trad normalisée, sans deux-points


You can enable strict mode for any part of your application. For example:
Vous pouvez activer le mode strict pour toutes les partie de votre application. Par exemple:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Vous pouvez activer le mode strict pour toutes les partie de votre application. Par exemple:
Vous pouvez activer le mode strict pour n’importe quelle partie de votre application. Par exemple :

`embed:strict-mode/enabling-strict-mode.js`

In the above example, strict mode checks will *not* be run against the `Header` and `Footer` components. However, `ComponentOne` and `ComponentTwo`, as well as all of their descendants, will have the checks.
Dans l'exemple ci-dessus, les vérifications du mode strict ne s'effectueront *pas* pour les composants `Header` et `Footer`. Cependant, les composants `ComponentOne` et `ComponentTwo`, ainsi que tous leurs descendants, seront vérifiés.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Dans l'exemple ci-dessus, les vérifications du mode strict ne s'effectueront *pas* pour les composants `Header` et `Footer`. Cependant, les composants `ComponentOne` et `ComponentTwo`, ainsi que tous leurs descendants, seront vérifiés.
Dans l'exemple ci-dessus, les vérifications du mode strict ne seront *pas* appliquées pour les composants `Header` et `Footer`. En revanche, les composants `ComponentOne` et `ComponentTwo`, ainsi que tous leurs descendants, seront vérifiés.

* [Detecting unexpected side effects](#detecting-unexpected-side-effects)
* [Detecting legacy context API](#detecting-legacy-context-api)
Actuellement, `StrictMode` est utilisé pour :
* [Identifier les composants incorporant un cycle de vie à risque](#identifying-unsafe-lifecycles)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* [Identifier les composants incorporant un cycle de vie à risque](#identifying-unsafe-lifecycles)
* [Identifier les composants utilisant des méthodes de cycle de vie dépréciées](#identifying-unsafe-lifecycles)

Pas le même sens que “unsafe“, mais en pratique toutes les LCM dangereuses / à risque sont dépréciées (et ce sont les seules à l'être, donc bon).


By intentionally double-invoking methods like the component constructor, strict mode makes patterns like this easier to spot.
En invoquant intentionnellement deux fois les méthodes comme le constructeur d'un composant, le mode strict facilite la détection de ces schémas.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
En invoquant intentionnellement deux fois les méthodes comme le constructeur d'un composant, le mode strict facilite la détection de ces schémas.
En invoquant volontairement deux fois les méthodes comme le constructeur d'un composant, le mode strict facilite la détection de ces schémas.


### Detecting legacy context API {#detecting-legacy-context-api}
### Détecter l'API dépréciée contexte {#detecting-legacy-context-api}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
### Détecter l'API dépréciée contexte {#detecting-legacy-context-api}
### Détecter l'API dépréciée de Contexte {#detecting-legacy-context-api}


The legacy context API is error-prone, and will be removed in a future major version. It still works for all 16.x releases but will show this warning message in strict mode:
L'API dépréciée contexte est source d'erreur, et sera supprimée dans une future version de React. Elle fonctionne toujours pour les versions 16.x mais le mode strict affichera ce message d'erreur:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
L'API dépréciée contexte est source d'erreur, et sera supprimée dans une future version de React. Elle fonctionne toujours pour les versions 16.x mais le mode strict affichera ce message d'erreur:
L'API dépréciée de Contexte est source d'erreur, et sera retirée dans une future version majeure de React. Elle fonctionne toujours dans les versions 16.x mais le mode strict affichera ce message d'avertissement :


The legacy context API is error-prone, and will be removed in a future major version. It still works for all 16.x releases but will show this warning message in strict mode:
L'API dépréciée contexte est source d'erreur, et sera supprimée dans une future version de React. Elle fonctionne toujours pour les versions 16.x mais le mode strict affichera ce message d'erreur:

![](../images/blog/warn-legacy-context-in-strict-mode.png)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
![](../images/blog/warn-legacy-context-in-strict-mode.png)
![Avertissement du mode strict à l'utilisation de l'API dépréciée de Contexte](../images/blog/warn-legacy-context-in-strict-mode.png)


![](../images/blog/warn-legacy-context-in-strict-mode.png)

Read the [new context API documentation](/docs/context.html) to help migrate to the new version.
Lisez la [documentation sur la nouvelle API contexte](/docs/context.html) pour faciliter la migration vers cette nouvelle version.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Lisez la [documentation sur la nouvelle API contexte](/docs/context.html) pour faciliter la migration vers cette nouvelle version.
Lisez la [documentation de la nouvelle API de Contexte](/docs/context.html) pour faciliter la migration vers cette nouvelle version.

@tdd tdd added Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR and removed Reviewing Un mainteneur qualifié est en train de composer la revue de code labels Mar 15, 2019
@tdd
Copy link
Collaborator

tdd commented Mar 16, 2019

@erkela pourrais-tu trouver le temps de traiter mes retours d'ici lundi midi ? pour info, si ce n'est pas le cas, j'auto-validerai les suggestions pour avancer.

@tdd
Copy link
Collaborator

tdd commented Mar 18, 2019

Salut @erkela

⚠️ 🚨 URGENT 🚨 ⚠️

Tu n'as pas autorisé l'accès aux mainteneurs sur ta PR. Je ne peux donc pas auto-accepter les suggestions.

Ta PR est gelée du coup, puisque tu ne traites pas mes retours et que je ne peux pas les auto-valider. Afin d'avancer, soit tu peux me donner les accès (vérifie aussi le statut de protection de tes branches, voire ajoute-moi en collaborateur à ton fork), soit je ferai demain une branche à moi basée sur la tienne, et je devrai me refader toutes les modifs à la main… 😒

@sheplu
Copy link
Collaborator

sheplu commented Mar 18, 2019

j'ai du temps libre demain @tdd donc je peux m'en occuper de reprendre la branche et d'appliquer les modifications à la main, ca permettra d'eviter que tu perdes du temps la dessus ;)

@tdd
Copy link
Collaborator

tdd commented Mar 19, 2019

@sheplu tu peux toujours oui, il te suffit de faire un git fetch origin pull/73/head:strict-mode pour avoir la branche strict-mode associée en local, après quoi tu devras te fader les applications, ce qui est très error-prone, donc sois bien vigilant 😉

Note que tu n'auras d'autre choix que de pondre une PR alternative, du coup 😒

@tdd
Copy link
Collaborator

tdd commented Mar 20, 2019

@sheplu tu as pu t'en occuper ?

@tdd tdd mentioned this pull request Mar 20, 2019
@tdd
Copy link
Collaborator

tdd commented Mar 20, 2019

Superseded by #121

@tdd tdd closed this Mar 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants