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

React dom common #584

Merged
merged 8 commits into from
Sep 12, 2023
Merged

React dom common #584

merged 8 commits into from
Sep 12, 2023

Conversation

linsolas
Copy link
Contributor

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
Copy link

github-actions bot commented Sep 10, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@linsolas linsolas marked this pull request as ready for review September 10, 2023 15:51
@tdd tdd self-assigned this Sep 10, 2023
@tdd tdd added Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR scope:required-pages Part of the required translations before we can ship translator:seasoned The author of that translation is a seasoned React docs translator labels Sep 10, 2023
@tdd
Copy link
Collaborator

tdd commented Sep 11, 2023

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
Copy link
Collaborator

tdd commented Sep 11, 2023

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
Copy link
Collaborator

tdd commented Sep 11, 2023

(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 tdd 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 Sep 11, 2023
@tdd tdd added Reviewed and removed Reviewing Un mainteneur qualifié est en train de composer la revue de code labels Sep 12, 2023
@tdd
Copy link
Collaborator

tdd commented Sep 12, 2023

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
Copy link
Collaborator

tdd commented Sep 12, 2023

(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
Copy link
Contributor Author

Hello @tdd,

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

@tdd tdd merged commit f695c46 into reactjs:main Sep 12, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reviewed scope:required-pages Part of the required translations before we can ship translator:seasoned The author of that translation is a seasoned React docs translator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants