Translation Add React to a website #72
Conversation
sheplu
added
the
Pending Review
|
Deploy preview for fr-reactjs ready! Built with commit 020a657 |
tdd
added
Reviewing
There was a problem hiding this comment.
Salut @Nab69 !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
Pour être tout à fait honnête, je suis surpris par cette trad, qui porte la marque d'une certaine fatigue pendant sa rédaction ; c'est très différent de tes PRs précédentes. Il y a surtout deux traits :
- Nombreux glissements sémantiques
- Nombreux alourdissements / ampoulages de phrase
C'est caractéristique d'un travail fatigué, ou d'un oubli du ton ciblé. On est ici sur la 2e page de la doc, juste après le Bien démarrer, il est essentiel que le ton décontracté, bienveillant et simplifié de la partie Tuto / Fondamentaux soit évident. Au lieu de quoi, ça regorge de formules presque académiques, façon white paper ou document légal.
Bref ! J'espère ne pas te vexer avec ça, personne n'est au top à chaque fois, moi le premier ! Mais du coup j'ai beaucoup de correctifs dans cette revue. J'aime à croire que tu n'auras pas d'ego mal placé et que tu percevras leur intérêt pour le résultat final !
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 !
|
|
||
| If you notice that you're getting comfortable with build tools and want them to do more for you, [the next section](/docs/create-a-new-react-app.html) describes some of the most popular and approachable toolchains. If not -- those script tags will do just fine! | ||
| Si vous vous sentez à l'aise avec les outils de compilation et souhaitez en apprendre plus, la [partie suivante](/docs/create-a-new-react-app.html) décrit les outils les plus populaires et les plus accessibles de la chaine de compilation. Si ce n'est pas le cas, les balises décrites précédemment fonctionneront à merveille ! |
There was a problem hiding this comment.
| Si vous vous sentez à l'aise avec les outils de compilation et souhaitez en apprendre plus, la [partie suivante](/docs/create-a-new-react-app.html) décrit les outils les plus populaires et les plus accessibles de la chaine de compilation. Si ce n'est pas le cas, les balises décrites précédemment fonctionneront à merveille ! | |
| Si vous vous sentez à l'aise avec les outils de compilation et souhaitez les utiliser davantage, la [prochaine section](/docs/create-a-new-react-app.html) décrit les boîtes à outils les plus populaires et les plus accessibles. Dans le cas contraire, les balises décrites précédemment fonctionneront à merveille ! |
tdd
added
Reviewed
and removed
Reviewing
Co-Authored-By: Nab69 <nab@codetrainer.fr>
Co-Authored-By: Nab69 <nab@codetrainer.fr>
Co-Authored-By: Nab69 <nab@codetrainer.fr>
Co-Authored-By: Nab69 <nab@codetrainer.fr>
Co-Authored-By: Nab69 <nab@codetrainer.fr>
Co-Authored-By: Nab69 <nab@codetrainer.fr>
Co-Authored-By: Nab69 <nab@codetrainer.fr>
Co-Authored-By: Nab69 <nab@codetrainer.fr>
|
@Nab69 raaaaaa par pitié pas de commits individuels par suggestion ! C'est dit clairement dans le résumé de review, passe par “add to batch” et fais un commit final unique 🙏 |
|
@Nab69 j'ai vu que tu as commencé à traiter les retours, auras-tu le temps de finir ça d'ici demain ? |
Co-Authored-By: Nab69 <nab@codetrainer.fr>
|
@Nab69 j'ai tenté de committer un lot de tes suggestions pour lesquelles j'étais OK, mais tu n'as pas donné les droits de commit sur ton fork aux mainteneurs du dépôt canonique 😉 Outre 2-3 points où nous débattons encore, il reste quelques retours à moi sur la fin que tu n'as pas encore traités ; tu auras le temps aujourd'hui ? |
Co-Authored-By: Nab69 <nab@codetrainer.fr>
|
@Nab69 le code de master a bougé pour ton fichier (y'avait une faute de frappe dans "unminified" dans la VO, je crois que c'est tout, merci de résoudre le conflit (via GitHub si tu veux), a priori en reprenant ton texte verbatim. |
|
@tdd ca doit être corrigé maintenant, ne reste que le dernier point pour lequel il faudrait qu'on trouve mieux mais comme on dit la nuit porte conseil ca va peut etre nous aider ;) |
|
@tdd petite question sur le final tweaks, j'ai vu que tu etais repassé sur pas mal d'espacements, les espaces insécables n'étaient pas passé correctement ? parce que j'ai utilisé les corrections proposés pour ça normalement, en passant par des editeurs de textes. Du coup je me dis que ma config est peut etre bizarre ou qu'il y a une difference entre ce que propose les editeurs et les md ? |
|
@Nab69 il y a quelques retouches de wording qui servent surtout à éviter des orphelines, et quelques espaces insécables en effet, sachant que je ne prends pas la peine de toutes les signaler en suggestions, c'est pénible à voir dans GitHub en fait, et je sais que mon final tweaks fait une repasse intégrale. |
No description provided.