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

introduction-jsx translation #10

Merged
merged 17 commits into from Feb 13, 2019
Merged

introduction-jsx translation #10

merged 17 commits into from Feb 13, 2019

Conversation

sylvhama
Copy link
Contributor

@tdd I would like to help too :) I guess I will start with Introducing JSX,

@sylvhama sylvhama changed the title WIP introduction-jsx translation [WIP] introduction-jsx translation Feb 10, 2019
@tesseralis tesseralis requested a review from tdd February 10, 2019 04:59
@tdd
Copy link
Collaborator

tdd commented Feb 10, 2019

@sylvhama quand tu seras prêt à recevoir une review, soit retire le WIP du titre (si tu as fini), soit fais-nous signe par commentaire.

@tdd
Copy link
Collaborator

tdd commented Feb 12, 2019

@sylvhama ça va ? Comment ça se passe ta trad ?

@sylvhama
Copy link
Contributor Author

@tdd En semaine j'ai pas trop de temps, je peux t'envoyer ça ce weekend ?

@tdd
Copy link
Collaborator

tdd commented Feb 12, 2019

@sylvhama oui sans souci ! Pense du coup à ne « prendre » des pages qu'en début de weekend, pour ne pas les verrouiller en semaine 😉

@sylvhama
Copy link
Contributor Author

@tdd je ferai plus attention les prochaines fois. Je pense que ma page est prête a être review.

@sylvhama sylvhama changed the title [WIP] introduction-jsx translation introduction-jsx translation Feb 13, 2019
@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 13, 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 @sylvhama !

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

J'espère ne pas me tromper, mais ça sent fort la première trad', j'ai raison ? Y'a absolument rien de mal à ça, il faut bien commencer quelque part, et la doc de React c'est un beau premier projet, en plus 😁

Mais du coup, forcément, y'a plein de retouches et reformulations. Ce qui est parfaitement normal et ne doit surtout pas te décourager. Tu vas automatiquement faire plein de progrès dès ta prochaine contribution, en internalisant les retours de celle-ci.

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é.

Si certaines appellent discussion, n'hésite pas, tu réponds, on en parle, on avance ensemble 🤝

J'ai aussi l'impression que tu as utilisé presque uniquement des espaces simples, et non insécables, avant les signes de ponctuation double (ainsi que des guillemets ASCII et non français). Une fois la PR finalisée, j'ajouterai les bonnes espaces dans un commit final, mais pour la prochaine fois, sache que tu peux les taper rapidement sur Mac avec Alt+Espace et sur Windows avec Alt+255.

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/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.md Outdated Show resolved Hide resolved
content/docs/introducing-jsx.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 Feb 13, 2019
tdd and others added 14 commits February 13, 2019 09:58
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Co-Authored-By: sylvhama <sylvhama@gmail.com>
Applied group of suggestions by tdd

Co-Authored-By: sylvhama <sylvhama@gmail.com>
@sylvhama
Copy link
Contributor Author

@tdd J'ai commit toutes tes suggestions

Quelques retouches oubliées, et les insécables
@tdd tdd merged commit 1b4d0a7 into reactjs:master Feb 13, 2019
@tdd
Copy link
Collaborator

tdd commented Feb 13, 2019

@sylvhama merci ! après une repasse visuelle qui a retouché deux-trois détails, c'est mergé !

@nicoclau
Copy link

nicoclau commented Nov 9, 2021

Embedding Expressions in JSX should be better translated as "Incorporer les Expressions dans JSX"

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

3 participants