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

Synthetic events #34

Merged
merged 6 commits into from
Feb 28, 2019
Merged

Synthetic events #34

merged 6 commits into from
Feb 28, 2019

Conversation

linsolas
Copy link
Contributor

Bonjour,

Voici ma proposition de traduction pour la page SyntheticEvent.

Quelques remarques / doutes :

  1. J'ai laissé wrapper tel quel, je ne sais pas s'il y a une traduction plus adaptée.
  2. Pour Pooling, j'ai utilisé partage (Event pooling => Partage d'événement). Pareil, si une meilleure traduction existe, j'adapterai.
  3. J'ai traduit les termes XXX Events par Événements de XXX (Keyboard Events par Événements de clavier). Je ne sais pas si Événements du clavier ou plus simplement Événements clavier est à préférer ?
  4. J'ai une hésitation sur la meilleure traduction pour cross-browsers : j'ai utilisé inter-navigateur, mais MDN privilégie navigateur croisé. Qu'en pensez-vous ?

@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 14, 2019
@tdd tdd self-assigned this Feb 14, 2019
Copy link
Collaborator

@sheplu sheplu left a comment

Choose a reason for hiding this comment

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

ca me semble bon
juste un doute entre l'utilisation de de ou du devant certains mots, les deux sont corrects mais de fait bizarre (du moins pour moi)
et changement de roulette en molette plutot


Your event handlers will be passed instances of `SyntheticEvent`, a cross-browser wrapper around the browser's native event. It has the same interface as the browser's native event, including `stopPropagation()` and `preventDefault()`, except the events work identically across all browsers.
Votre gestionnaire d'événement va recevoir des instances de `SyntheticEvent`, un wrapper inter-navigateur autour de l'événement natif du navigateur. Il dispose de la même interface que l'événement natif du navigateur, ce qui inclut `stopPropagation()` et `preventDefault()`, à ceci près que les événements fonctionnent de façon identique sur tous les navigateurs.
Copy link
Collaborator

Choose a reason for hiding this comment

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

gestionnaire d'événement va -> gestionnaire d'événements va
pluriel je pense parce qu'il gere plusieurs events

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'est vrai que la question se pose, mais j'ai pris le Glossaire comme référence qui utilise Gestionnaire d’événement. Cela dit, la phrase en anglais est au pluriel (your event handlers), je vais corriger la phrase.


- [Événements de presse-papiers](#clipboard-events)
- [Événements de composition](#composition-events)
- [Événements de clavier](#keyboard-events)
Copy link
Collaborator

Choose a reason for hiding this comment

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

du clavier ?

- [Événements de composition](#composition-events)
- [Événements de clavier](#keyboard-events)
- [Événements de focus](#focus-events)
- [Événements de formulaire](#form-events)
Copy link
Collaborator

Choose a reason for hiding this comment

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

du formulaire ?

- [Événements de focus](#focus-events)
- [Événements de formulaire](#form-events)
- [Événements de souris](#mouse-events)
- [Événements de pointeur](#pointer-events)
Copy link
Collaborator

Choose a reason for hiding this comment

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

du pointeur ?

- [Événements de sélection](#selection-events)
- [Événements d'interaction tactile](#touch-events)
- [Événements visuels](#ui-events)
- [Événements de roulette](#wheel-events)
Copy link
Collaborator

Choose a reason for hiding this comment

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

de la roulette ?


```
onFocus onBlur
```

These focus events work on all elements in the React DOM, not just form elements.
Ces événements de focus fonctionnent sur tous les éléments du DOM de React, et pas uniquement sur les éléments de formulaire.
Copy link
Collaborator

Choose a reason for hiding this comment

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

du formulaire


```javascript
DOMEventTarget relatedTarget
```

* * *

### Form Events {#form-events}
### Événements de formulaire {#form-events}
Copy link
Collaborator

Choose a reason for hiding this comment

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

du formulaire


* * *

### Mouse Events {#mouse-events}
### Événements de souris {#mouse-events}
Copy link
Collaborator

Choose a reason for hiding this comment

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

de la souris

@@ -216,20 +216,20 @@ boolean shiftKey

* * *

### Pointer Events {#pointer-events}
### Événements de pointeur {#pointer-events}
Copy link
Collaborator

Choose a reason for hiding this comment

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

du pointeur

@@ -302,15 +302,15 @@ DOMAbstractView view

* * *

### Wheel Events {#wheel-events}
### Événements de roulette {#wheel-events}
Copy link
Collaborator

Choose a reason for hiding this comment

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

de la molette

@linsolas
Copy link
Contributor Author

Bonsoir @sheplu

Merci pour ta relecture. J'ai intégré tes corrections. J'ai la même hésitation pour du / de, j'ai changé selon ta suggestion. On verra ce qu'en pense @tdd ;)

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for fr-reactjs ready!

Built with commit c9e9675

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

@tdd tdd added Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Feb 27, 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 @linsolas !

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

Pas mal de retouches tout de même, certains points n'étant apparemment pas clairs pour toi, sur le fond, quand tu as traduit. On trouve aussi diverses « infractions » aux MP / Glossaire, sans doute parce que c'est une de tes premières contribs et que certains trucs ont été normalisés après l'envoi de cette PR.

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 !

content/docs/reference-events.md Outdated Show resolved Hide resolved
content/docs/reference-events.md Outdated Show resolved Hide resolved
content/docs/reference-events.md Outdated Show resolved Hide resolved
content/docs/reference-events.md Outdated Show resolved Hide resolved
content/docs/reference-events.md Outdated Show resolved Hide resolved
content/docs/reference-events.md Outdated Show resolved Hide resolved
@@ -244,33 +244,33 @@ string pointerType
boolean isPrimary
```

A note on cross-browser support:
Une remarque concernant le support inter-navigateur :
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
Une remarque concernant le support inter-navigateur :
Une remarque concernant la pris en charge sur tous les navigateurs :

« inter-navigateur » n'est décidément pas souhaitable ; je n'ai d'ailleurs jamais vu ce terme nulle part…?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

J'ai hésité et comme je l'avais trouvé sur une page de MDN je me suis laissé convaincre... Je n'aurais pas dû ;)

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Une remarque concernant le support inter-navigateur :
Une remarque concernant la prise en charge sur tous les navigateurs :

Avec une "e" à "prise" c'est mieux 😄

content/docs/reference-events.md Outdated Show resolved Hide resolved
content/docs/reference-events.md Outdated Show resolved Hide resolved
content/docs/reference-events.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 28, 2019
Co-Authored-By: linsolas <linsolas@gmail.com>
@linsolas
Copy link
Contributor Author

Merci pour ta relecture @tdd.

@tdd tdd merged commit 9c2da39 into reactjs:master Feb 28, 2019
@linsolas linsolas deleted the synthetic-events branch February 28, 2019 15:31
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.

4 participants