Translate lifting-state-up

[thib92]

Translate the page "Lifting State Up".

[thib92]

@tdd WIP, fichier assez gros, je finis ce week-end.
Est-ce que tu as un tip pour s'assurer de mettre des espaces insécables pour les guillemets à la française ? Mon VSCode les affiche comme des espaces, donc difficile de ne pas en manquer.

[Nab69]

@tdd WIP, fichier assez gros, je finis ce week-end.
Est-ce que tu as un tip pour s'assurer de mettre des espaces insécables pour les guillemets à la française ? Mon VSCode les affiche comme des espaces, donc difficile de ne pas en manquer.

@thib92 D'après le site de delicious insight et sa liste de plugins utiles (on se demande qui est derrière un site qui parle d'espaces insécables 😄 ), il y en a un pour ça : https://marketplace.visualstudio.com/items?itemName=possan.nbsp-vscode (que je vais dl aussi d'ailleurs)

[netlify]

Deploy preview for fr-reactjs ready!

Built with commit 6a4dd91

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

[thib92]

Ready to review 💪

[JeremiePat]

Merci pour cette excellente traduction :) J'ai proposer quelques changement de forme mais globalement, c'est nickel.

Tu notera que j'ai corriger "prop" partout pour qu'on l'utilise au féminin (prop est le raccourcis de propriété qui est un nom féminin). @tdd qu'est-ce que tu en pense ? Ça vaudrait peut-être le coup de le préciser dans le glossaire.

[tdd]

@JeremiePat @thib92 prop est clairement féminin, cf reste des trads approuvées, overreacted, etc.

[thib92]

Entendu, c’est une question que je me suis posée.
Merci pour la correction !

[sheplu]

(j'ai juste un gros doute sur le genre de prop, tu as du feminin et masculin qui changent ou bien des accords qui une fois ce font au masculin l 'autre fois au feminin)

my bad, a croire que je suis parti d'une page non actualise donc j ai pas vu la review de jeremie :/

pas actualise sur la review de jeremie

[tdd]

Salut @thib92 !

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

64 blocs de modifs, 60 reviews : ne te laisse pas impressionner 😉 — ce sont pour la plupart les mêmes points (« entrée » qui aurait dû être 99% du temps « champ », etc.).

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 !

[thib92]

@tdd J'ai mis presque toutes tes suggestions, j'ai juste commenté les deux remplacements de "on" à "nous". A toi de voir 😄

[tdd]

@thib92 je vais quand même insister pour les « nous » si tu le permets (je peux le faire en repasse visuelle finale), pour deux raisons :

1. En pratique « nous » n'est pas plus léger que « on », qui est souvent maladroit car il peut créer un hiatus. Il est par ailleurs moins élégant.
2. Il y a, comme je l'indiquais, une différence d'emploi. « On » est impersonnel, il correspond davantage au pronom “One” en anglais. Il peut être utilisé pour décrire un comportement / aspect général. À l'inverse, « Nous » est personnel, on le préfère quand on suit un déroulé qui emploie “we” ou “you”, car il favorise l’implication du lectorat.

Encore merci !

[tdd]

Rappel : casse française. J'ajusterai en repasse visuelle finale.

[tdd]

@thib92 la repasse visuelle a aussi mis en avant l'absence fréquente d'insécables devant les signes de ponctuation doubles. Penses-y la prochaine fois, dans l'idéal ! 😉

[thib92]

@tdd J'ai essayé de faire attention mais je n'ai pas encore le réflexe 😉
Merci pour la review !