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

Translation of the Tutorial #81

Merged
merged 5 commits into from Mar 5, 2019
Merged

Translation of the Tutorial #81

merged 5 commits into from Mar 5, 2019

Conversation

tdd
Copy link
Collaborator

@tdd tdd commented Mar 2, 2019

@JeremiePat j'aurais vraiment besoin rapidement de ta revue là-dessus, pour boucler 100% du Core ASAP (ça me saoule qu'on n'y soie pas encore, j'avoue).

(Et au passage, j'attends ton traitement de ma revue pour ta PR #16 sur l'API React 😉)

@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Mar 2, 2019
@tdd tdd self-assigned this Mar 2, 2019
@tdd tdd requested a review from JeremiePat March 2, 2019 21:25
@netlify
Copy link

netlify bot commented Mar 2, 2019

Deploy preview for fr-reactjs ready!

Built with commit 6958371

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

@sheplu
Copy link
Collaborator

sheplu commented Mar 2, 2019

je regardais ta PR pour faire un debut de review, et simple question sur le double espace en debut de phrase (espace dactylographique). Parce que sauf erreur, tu es le seul à l'avoir utilisé. On garde la double convention ?

@tdd
Copy link
Collaborator Author

tdd commented Mar 3, 2019

@sheplu les vieilles habitudes ont la vie dure... Ça remonte aux temps des mails texte, de Usenet, IRC, etc. Convention dactylo monospace.

Ça n'a absolument rien d'obligatoire et ne change rien au rendu HTML. Tu peux donc l'ignorer 😉, même si ça fait souvent tiquer@JeremiePat.

Copy link
Contributor

@forresst forresst left a comment

Choose a reason for hiding this comment

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

Je n'ai pas eu le temps de tout relire (seulement 20% environ), mais j'ai quelques remarques. Si j'ai du temps cet après midi j’essaierai de relire la suite. "Désolé, je dois aller changer la couche de l'étudiant ..."

content/tutorial/tutorial.md Show resolved Hide resolved
content/tutorial/tutorial.md Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
Merci @forresst!

Co-Authored-By: tdd <tdd@tddsworld.com>
Copy link
Contributor

@forresst forresst left a comment

Choose a reason for hiding this comment

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

J'ai continuer la relecture, c'est un travail assez long, j'ai fait environ 75%. Il faut l'avouer c'est du très bon boulot : 3 petites remarques (histoire de discuter un peu 😄)

content/tutorial/tutorial.md Show resolved Hide resolved
content/tutorial/tutorial.md Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
Co-Authored-By: tdd <tdd@tddsworld.com>
Copy link
Contributor

@forresst forresst left a comment

Choose a reason for hiding this comment

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

Voilà, j'ai fini ma relecture, désolé encore pour ce découpage de relecture en 3 morceaux. Quelques remarques sur la fin. @tdd, c'est un plaisir de te relire

content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
Co-Authored-By: tdd <tdd@tddsworld.com>
@JeremiePat JeremiePat self-assigned this Mar 4, 2019
@JeremiePat JeremiePat 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 4, 2019
Copy link
Collaborator

@JeremiePat JeremiePat left a comment

Choose a reason for hiding this comment

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

Et voila :) Quelque point de détail à revoir, mais c'est bien parce que je chipote 🙂

content/tutorial/tutorial.md Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved

If you need to review JavaScript, we recommend reading [this guide](https://developer.mozilla.org/en-US/docs/Web/JavaScript/A_re-introduction_to_JavaScript). Note that we're also using some features from ES6 -- a recent version of JavaScript. In this tutorial, we're using [arrow functions](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions), [classes](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Classes), [`let`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let), and [`const`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/const) statements. You can use the [Babel REPL](babel://es5-syntax-example) to check what ES6 code compiles to.
Si vous avez besoin de réviser votre JavaScript, nous vous conseillons la lecture de [ce guide](https://developer.mozilla.org/fr/docs/Web/JavaScript/Une_réintroduction_à_JavaScript). Remarquez que nous utilisons aussi certains aspects d’ES6—une version récente de JavaScript. Dans ce tutoriel, on utilise les [fonctions fléchées](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Fonctions/Fonctions_fléchées), les [classes](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Classes), et les instructions [`let`](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Instructions/let), et [`const`](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Instructions/const). Vous pouvez utiliser la [REPL Babel](babel://es5-syntax-example) pour examiner le résultat de la compilation de code ES6.
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
Si vous avez besoin de réviser votre JavaScript, nous vous conseillons la lecture de [ce guide](https://developer.mozilla.org/fr/docs/Web/JavaScript/Une_réintroduction_à_JavaScript). Remarquez que nous utilisons aussi certains aspects d’ES6une version récente de JavaScript. Dans ce tutoriel, on utilise les [fonctions fléchées](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Fonctions/Fonctions_fléchées), les [classes](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Classes), et les instructions [`let`](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Instructions/let), et [`const`](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Instructions/const). Vous pouvez utiliser la [REPL Babel](babel://es5-syntax-example) pour examiner le résultat de la compilation de code ES6.
Si vous avez besoin de réviser votre JavaScript, nous vous conseillons la lecture de [ce guide](https://developer.mozilla.org/fr/docs/Web/JavaScript/Une_réintroduction_à_JavaScript). Remarquez que nous utilisons aussi certains aspects d’ES6 – une version récente de JavaScript. Dans ce tutoriel, on utilise les [fonctions fléchées](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Fonctions/Fonctions_fléchées), les [classes](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Classes), et les instructions [`let`](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Instructions/let), et [`const`](https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Instructions/const). Vous pouvez utiliser la [REPL Babel](babel://es5-syntax-example) pour examiner le résultat de la compilation de code ES6.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Bon, même si je ne suis pas un grand fan des tirets pour les incises, on va dire que bon, ok, c'est une pratique de plus en plus courante. Par contre, dans ce cas, il est recommandé d'utiliser un demi-cadratin (le cadratin est réservé à au changement de parole dans les dialogues) entouré d'espaces fines insécables (donc sur le web, on se contentera de simples espaces insécables). Sous Mac, cadratin: alt + - demi-cadratin: alt + maj + -

content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
@JeremiePat JeremiePat added Reviewed and removed Reviewing Un mainteneur qualifié est en train de composer la revue de code labels Mar 4, 2019
We still need to crystallize the em-dash policy

Co-Authored-By: tdd <tdd@tddsworld.com>
@tdd tdd merged commit ee0f40e into master Mar 5, 2019
@tdd tdd deleted the tutorial branch March 5, 2019 10:27
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.

None yet

5 participants