Guide: dom-events.md #19

mdartic · 2017-11-02T22:32:00Z

Traduction de la partie dom-events, merci de relire !

Première traduction

MachinisteWeb

Le gros de ce que j'ai vu c'est la ponctuation FR, ne pas traduire les propriétés, méthode et nom de fichir et utiliser la graphie post 1990 :)

Merci pour la trad !

MachinisteWeb · 2017-11-03T15:59:47Z

docs/en/guides/README.md

@@ -2,7 +2,7 @@

 * [Commencer](./getting-started.md)
 * [Astuces](./common-tips.md)
-* [Mouse, Key and other DOM Events (EN)](guides/dom-events.md)
+* [Mouse, Key and other DOM Events (EN)](./dom-events.md)


Tu peux retirer le EN :) Et traduire la ligne :)

MachinisteWeb · 2017-11-03T16:00:41Z

docs/en/guides/dom-events.md

@@ -1,16 +1,17 @@
-# Testing Key, Mouse and other DOM events
+# Tester le clavier, la souris et les autres événements DOM


On utilise la graphie post 1990. Donc

évèments

MachinisteWeb · 2017-11-03T16:01:15Z

docs/en/guides/dom-events.md


-The `Wrapper` expose a `trigger` method. It can be used to trigger DOM events.
+
+Le `Wrapper` expose une méthode `trigger` method. Elle peut être utilisée pour déclencher des événements du DOM.


~~method~~.

MachinisteWeb · 2017-11-03T16:01:42Z

docs/en/guides/dom-events.md


 ```js
 const wrapper = mount(MyButton)

 wrapper.trigger('click')
 ```

-You should be aware, that find returns a wrapper as well. Assuming `MyComponent` contains a button, the following code clicks the button.
+Vous devez être au courant que la méthode find retourne un wrapper aussi. Partant du principe que `MyComponent` contient un bouton, le code suivant clique sur le bouton.


`find`

(Tu peux PR l'original)

MachinisteWeb · 2017-11-03T16:02:34Z

docs/en/guides/dom-events.md

@@ -20,9 +21,9 @@ wrapper.find('button').trigger('click')

 ## Options

-The trigger method takes an optional `options` object. The properties in the `options` object are added to the Event.
+La méthode trigger prend en paramètre optionnel l'objet `options`. Les propriétés de l'objet `options` sont ajoutées à l'événement Event.


La méthode `trigger`

sont ajoutées à l'évènement Event.

ou

sont ajoutées à l'évènement.

MachinisteWeb · 2017-11-03T16:12:07Z

docs/en/guides/dom-events.md

-  it('Quantity is zero by default', () => {
-    const wrapper = mount(QuantityComponent)
-    expect(wrapper.vm.quantity).to.equal(0)
+describe('Tests événement clavier', () => {


évènement

MachinisteWeb · 2017-11-03T16:12:19Z

docs/en/guides/dom-events.md

-    expect(wrapper.vm.quantity).to.equal(0)
+describe('Tests événement clavier', () => {
+  it('La quantité est zéro par défaut', () => {
+    const wrapper = mount(QuantiteComponent)


Remettre en EN et le reste du code (garder en FR ce qui est dans it(

MachinisteWeb · 2017-11-03T16:13:03Z

docs/en/guides/dom-events.md

-  it('Escape sets quantity to 0', () => {
-    const wrapper = mount(QuantityComponent)
-    wrapper.vm.quantity = 5
+  it('La touche Echap positionne la quantité à 0', () => {


La touche Échap

MachinisteWeb · 2017-11-03T16:13:33Z

docs/en/guides/dom-events.md

  })
 })

 ```

 **Limitations**

-A key name after the dot `keydown.up` is translated to a `keyCode`. This is supported for the following names:
+Un nom de touche après le point `keydown.up` est traduit vers un `keyCode`. Cela est supporté pour les noms suivant:


les noms suivant : (espace avant les :)

MachinisteWeb · 2017-11-03T16:13:44Z

docs/en/guides/dom-events.md


 * enter, tab, delete, esc, space, up, down, left, right

 ## Important

-vue-test-utils triggers event synchronously. Consequently, `vue.nextTick` is not required.
+vue-test-utils déclenche les événements de façon synchrone. Par conséquent, `vue.nextTick` n'est pas requis.


évènements

rspt

Niveau orthographe c'est plutôt ok pour moi! Juste événements à changer pour évènements comme souligné par @haeresis

mdartic · 2017-11-05T17:51:45Z

Merci @haeresis & @rspt
J'ai pris en compte tes demandes @haeresis , mais j'ai encore le doute sur le yes/no.

MachinisteWeb · 2017-11-05T18:06:08Z

Le paramètre de this.callMe('yes') est une chaîne de caractères, donc on le traduit.

MachinisteWeb · 2017-11-05T18:23:41Z

T'en fais pas, je m'en occupe après merge, j'ai pas le contexte complet sur mon mobile, j'imagine qu'en fait c'est plus subtile ?!

Yaty

Voilà ma review, bonne trad :)

Yaty · 2017-11-05T20:19:59Z

docs/en/guides/dom-events.md


 ```js
 const wrapper = mount(MyButton)

 wrapper.trigger('click')
 ```

-You should be aware, that find returns a wrapper as well. Assuming `MyComponent` contains a button, the following code clicks the button.
+Vous devez être au courant que la méthode `find` retourne un wrapper aussi. Partant du principe que `MyComponent` contient un bouton, le code suivant clique sur le bouton.


retourne aussi un wrapper.

En partant du principe

Yaty · 2017-11-05T20:20:49Z

docs/en/guides/dom-events.md


-**Component under test**
+**Componsant à tester**


Yaty · 2017-11-05T20:22:24Z

docs/en/guides/dom-events.md

-describe('Click event', () => {
-  it('Click on yes button calls our method with argument "yes"', () => {
+describe('Évènement click', () => {
+  it('Cliquer sur le bouton oui appelle notre méthode avec l\'argument "yes"', () => {


"Cliquer sur le bouton oui appelle notre méthode avec l'argument 'yes'", on avait fait ça pour les autres fichiers afin d'éviter les backslashs

MachinisteWeb · 2017-11-06T10:09:33Z

Je te laisse prendre les retours de @Yaty, @mdartic et on pourra merge :)

mdartic · 2017-11-06T11:15:29Z

Done
J'ai traduit du coup les this.callMe('yes/no') en this.callMe('oui/non'), mais ça me paraît un peu cavalier dans le sens où les tests sont à la fois en anglais & français.
On a une classe css yes qui est utilisée pour cibler le bouton, et on valide que la méthode est bien appelée avec oui... ça vous paraît pas étrange ?

MachinisteWeb · 2017-11-06T15:19:19Z

Pour moi c'est tout bon, le libélé du bouton cliqué est bien « Oui » <button class="yes" @click="callYes">Oui</button>

MachinisteWeb · 2017-11-06T15:54:26Z

Super @Yaty, @mdartic et @rspt

La pratie Guides est en PR ;)
— vuejs#159

Partie API maintenant :)

Guide: dom-events.md

d9435bb

Première traduction

mdartic requested review from MachinisteWeb and Yaty November 2, 2017 22:32

MachinisteWeb mentioned this pull request Nov 2, 2017

Pages en cours de traduction #1

Open

Update README.md

befc38d

MachinisteWeb requested changes Nov 3, 2017

View reviewed changes

MachinisteWeb requested a review from rspt November 3, 2017 16:15

MachinisteWeb assigned MachinisteWeb and mdartic and unassigned MachinisteWeb Nov 3, 2017

MachinisteWeb added the Relecture nécessaire label Nov 3, 2017

rspt approved these changes Nov 5, 2017

View reviewed changes

mdartic added 2 commits November 5, 2017 18:33

Update dom-events.md

f1b13ae

Update README.md

3a7a08e

Yaty reviewed Nov 5, 2017

View reviewed changes

Prise en compte des remarques de @yati & @haeresis

cebe2a6

MachinisteWeb merged commit 95fa5db into working Nov 6, 2017

MachinisteWeb deleted the guide-dom-events branch November 6, 2017 15:19

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Guide: dom-events.md #19

Guide: dom-events.md #19

mdartic commented Nov 2, 2017

MachinisteWeb left a comment

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

MachinisteWeb Nov 3, 2017

rspt left a comment

mdartic commented Nov 5, 2017

MachinisteWeb commented Nov 5, 2017

MachinisteWeb commented Nov 5, 2017

Yaty left a comment

Yaty Nov 5, 2017

Yaty Nov 5, 2017

Yaty Nov 5, 2017

MachinisteWeb commented Nov 6, 2017

mdartic commented Nov 6, 2017 •

edited

Loading

MachinisteWeb commented Nov 6, 2017

MachinisteWeb commented Nov 6, 2017

		@@ -1,16 +1,17 @@
		# Testing Key, Mouse and other DOM events
		# Tester le clavier, la souris et les autres événements DOM


		The `Wrapper` expose a `trigger` method. It can be used to trigger DOM events.

		Le `Wrapper` expose une méthode `trigger` method. Elle peut être utilisée pour déclencher des événements du DOM.

Guide: dom-events.md #19

Guide: dom-events.md #19

Conversation

mdartic commented Nov 2, 2017

MachinisteWeb left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rspt left a comment

Choose a reason for hiding this comment

mdartic commented Nov 5, 2017

MachinisteWeb commented Nov 5, 2017

MachinisteWeb commented Nov 5, 2017

Yaty left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MachinisteWeb commented Nov 6, 2017

mdartic commented Nov 6, 2017 • edited Loading

MachinisteWeb commented Nov 6, 2017

MachinisteWeb commented Nov 6, 2017

mdartic commented Nov 6, 2017 •

edited

Loading