React dom common - Githubissues

reactjs / fr.react.dev

Version française du site de documentation officiel de React

https://fr.react.dev

Creative Commons Attribution 4.0 International

98 stars 162 forks source link

React dom common #584

Closed linsolas closed 1 year ago

linsolas commented 1 year ago

Quelques remarques :

je n'ai pas encore fait ma propre lecture (d'où le draft).
j'ai mis tous les liens MDN vers la VF, même quand la page n'existe pas (ex. https://developer.mozilla.org/fr/docs/Web/API/MouseEvent/ctrlKey). MDN propose dans ce cas le lien vers la page en anglais, donc le jour où ce sera traduit, on sera bon !
vu la page, j'ai sans doute manquer de cohérence sur la longueur, désolé !
certains termes sont pas forcément faciles à traduire, je me suis parfois inspiré de la doc de MDN mais ce n'était pas toujours possible, donc ne pas hésiter à me rectifier !

github-actions[bot] commented 1 year ago

Size changes

## 📦 Next.js Bundle Analysis for react-dev This analysis was generated by the [Next.js Bundle Analysis action](https://github.com/hashicorp/nextjs-bundle-analysis). 🤖 This PR introduced no changes to the JavaScript bundle! 🙌

tdd commented 1 year ago

Coucou @linsolas !

Vu la nature de la page et certains choix (le "/fr" inexistant est par exemple moins utile que retirer purement et simplement le code de langue, qui redirige en auto selon la dispo et la langue détectée), si ça te va je vais directement retravailler et finaliser sur un commit de mon côté. Tu pourras en regarder le diff pour me confirmer que ça te va, après quoi je fusionnerai.

tdd commented 1 year ago

Je sais pas si t'as vu ces !@# d'auteurs VO viennent de rajouter 2 pages (heureusement assez courtes) dans la section Référence 🤣. Je les ai intégrées à notre to-do, ce qui nous ramène à 97% ; quand l'espagnol le fera, ils retomberont à 98%. J'ai prévu de les traduire ce jour ou demain, ce qui veut dire qu'on risque d'être 100% et eux pas à un moment.

Concours de cojones stupide j'en conviens, mais ça m'arracherait tout de même un sourire 😁

tdd commented 1 year ago

(Au passage tu devrais configurer ton fork pour retirer les branches sur la fusion des PR, tu as du cruft sur ton remote là)

tdd commented 1 year ago

Yo @linsolas voici un lien pour consulter les modifs de mes commits de review. Je te laisse me faire un retour rapide avec tes questions et suggestions éventuelles. Si c'est bon pour toi, on merge !

tdd commented 1 year ago

(Si tu vois parfois des formules un peu plus longues, surtout pour le "final review pass", ce sont généralement des astuces pour éviter des mots orphelins en début de ligne.)

linsolas commented 1 year ago

Hello @tdd,

Je viens de relire tes modifications, c'est bon pour moi !