Skip to content

events.md #27

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

Closed
wants to merge 3 commits into from
Closed

events.md #27

wants to merge 3 commits into from

Conversation

Ti-Bretin
Copy link

Hey guy's, J'ai essayé de faire de mon mieux pour la traduction de cette page. C'est la premiere fois que je traduis une page technique et je ne connais pas encore vuejs. En esperant que cela convienne je vous souhaite une bonne journée.

Note : J'ai mis un peu de temps a trouver les instructions pour savoir par ou commencer...Peut etre que je ne suis pas habitué, ou peut être qu'il serait mieux de placer les instructions dans le readme...thx

@MachinisteWeb MachinisteWeb changed the title Traduction vuesjs page documentation events.md Traduction events.md Feb 7, 2017
Copy link

@yann-yinn yann-yinn left a comment

Choose a reason for hiding this comment

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

Merci pour la traduction ! Voici mes suggestions, je termine plus tard la fin. Quelques remarques, ne pas hésiter à poser des questions auparavant sur une issue pour bien comprendre le sens si il y a trop de doutes sur des passages, ça évite un surplus de boulot lors de la review (et dans l'idéal attention aux typos :-p )


For example:
Par example:

Choose a reason for hiding this comment

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

example => exemple


``` html
<div id="example-2">
<!-- `greet` is the name of a method defined below -->
<!-- `greet` est le nom de la méthode defini ce-dessous-->

Choose a reason for hiding this comment

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

définie


The logic for many event handlers will be more complex though, so keeping your JavaScript in the value of the `v-on` attribute simply isn't feasible. That's why `v-on` can also accept the name of a method you'd like to call.
La logique pour beaucoup des gestionnaires d'événements sera très certainement plus compliqué, par consequence, garder vos valeurs dans l'attribu `v-on` ne sera pas possible. C'est pour cette raison que `v-on` peut également accepter que vous nommiez vos méthodes comme vous le souhaitez.

Choose a reason for hiding this comment

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

"sera cependant plus complexe" à la place de "sera très certainement plus compliqué" (plus proche de l'original je crois)

Choose a reason for hiding this comment

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

"pour beaucoup des" => "pour beaucoup de" ( me semble plus français)

Choose a reason for hiding this comment

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

consequence => conséquence

Choose a reason for hiding this comment

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

De gros contre-sens ici :
"garder vos valeurs dans l'attribu v-on ne sera pas possible" =>
"garder votre javascript dans la valeur de l'attribut v-on ne sera tout simplement pas possible. C'est pourquoi v-on peut aussi accepter le nom d'une méthode que vous voudriez appeler."

Copy link
Member

Choose a reason for hiding this comment

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

JavaScript


The logic for many event handlers will be more complex though, so keeping your JavaScript in the value of the `v-on` attribute simply isn't feasible. That's why `v-on` can also accept the name of a method you'd like to call.
La logique pour beaucoup des gestionnaires d'événements sera très certainement plus compliqué, par consequence, garder vos valeurs dans l'attribu `v-on` ne sera pas possible. C'est pour cette raison que `v-on` peut également accepter que vous nommiez vos méthodes comme vous le souhaitez.

Choose a reason for hiding this comment

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

attribu => attribut

@@ -4,16 +4,16 @@ type: guide
order: 9
---

## Listening to Events
## Ecoute des événements

Choose a reason for hiding this comment

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

ne pas oublier de traduire le "title" tout en haut du fichier "Gestion des évènements"


It is a very common need to call `event.preventDefault()` or `event.stopPropagation()` inside event handlers. Although we can do this easily inside methods, it would be better if the methods can be purely about data logic rather than having to deal with DOM event details.
C'est un besoin courant que de faire appel ā `event.preventDefault()` ou `event.stopPropagation()` à l'intérieur d'une déclaration d'événements. Bien que nous puissions realiser ceci aisément à l'intérieur de "Methods", il aurait été préférable que "Methods" reste purement dedié à la logique des données au lieu d'avoir à prendre en charge les éléments du DOM>

Choose a reason for hiding this comment

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

"d'une déclaration de gestionnaire d'évènement"
realiser => réaliser
"il aurait été préférable" => il serait préférable


To address this problem, Vue provides **event modifiers** for `v-on`. Recall that modifiers are directive postfixes denoted by a dot.
Pour resoudre ce problème, Vue propose des modifieurs d'événement à `v-on`. Evoquer ces modifieurs se fait grâce aux extensions d'instruction, ceux-ci commençant par un point.

Choose a reason for hiding this comment

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

modifieurs => modificateur

@@ -171,87 +171,86 @@ To address this problem, Vue provides **event modifiers** for `v-on`. Recall tha
- `.once`

``` html
<!-- the click event's propagation will be stopped -->
<a v-on:click.stop="doThis"></a>
<!-- la propagation de l'événement du clic sera stoppé -->

Choose a reason for hiding this comment

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

stoppée


<!-- modifiers can be chained -->
<a v-on:click.stop.prevent="doThat"></a>
<!-- les modifieurs peuvent être chainés -->

Choose a reason for hiding this comment

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

modificateurs

Unlike the other modifiers, which are exclusive to native DOM events, the `.once` modifier can also be used on [component events](components.html#Using-v-on-with-Custom-Events). If you haven't read about components yet, don't worry about this for now.

## Key Modifiers
Au contraire des autres modifieurs, qui sont exclusifs aux événement natif du DOM, le modifieur `.once` peut également être utilisé pour les [component events](components.html#Using-v-on-with-Custom-Events). Si vous n'avez pas encore lu la section concernant les composants, ne vous en inquitez pas pour le moment.

Choose a reason for hiding this comment

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

modificateurs
natif => natifs
traduire components events
inquitez => inquiétez

Copy link
Member

@MachinisteWeb MachinisteWeb left a comment

Choose a reason for hiding this comment

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

Merci Ti-Bretin pour ta traduction. Tu vas avoir des choses à revoir. Je referais une lecture quand tu auras rectifier les points soulevé.

La traduction de « modifier » est normalement « modificateur ».


<p class="tip">**Cette page est en cours de traduction française. Revenez une autre fois pour lire une traduction achevée ou [participez à la traduction française ici](https://github.com/vuejs-fr/vuejs.org).**</p>We can use the `v-on` directive to listen to DOM events and run some JavaScript when they're triggered.
Nous pouvons utiliser l'instruction `v-on` pour écouter les événements du DOM afin d'éxécuter des scripts Javascript lorsque ces événements surviennent.
Copy link
Member

Choose a reason for hiding this comment

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

JavaScript


For example:
Par example:
Copy link
Member

Choose a reason for hiding this comment

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

Par exemple :

exemple prend un « e » en français.

un espace en français entre (exemple et « : »).


``` html
<div id="example-1">
<button v-on:click="counter += 1">Add 1</button>
<p>The button above has been clicked {{ counter }} times.</p>
<p>Le boutton ci-dessus à été cliqué {{ counter }} fois.</p>
Copy link
Member

Choose a reason for hiding this comment

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

bouton

Copy link
Member

Choose a reason for hiding this comment

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

a

(verbe)

@@ -25,12 +25,12 @@ var example1 = new Vue({
})
```

Result:
Resultat:
Copy link
Member

Choose a reason for hiding this comment

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

Résultat :


{% raw %}
<div id="example-1" class="demo">
<button v-on:click="counter += 1">Add 1</button>
<p>The button above has been clicked {{ counter }} times.</p>
<p>Le boutton ci-dessus à été cliqué {{ counter }} fois.</p>
Copy link
Member

Choose a reason for hiding this comment

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

bouton

et

à


You might be concerned that this whole event listening approach violates the good old rules about "separation of concerns". Rest assured - since all Vue handler functions and expressions are strictly bound to the ViewModel that's handling the current view, it won't cause any maintenance difficulty. In fact, there are several benefits in using `v-on`:
Vous pouriez être preoccupé que tous ces événements d'écoutes viole la bonne vieille règle de la separation des problèmes.
Copy link
Member

@MachinisteWeb MachinisteWeb Feb 7, 2017

Choose a reason for hiding this comment

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

séparation des préoccupations

(je sais, c'est bizarre, mais c'est la traduction la plus courante et validée ici)


1. It's easier to locate the handler function implementations within your JS code by simply skimming the HTML template.
1. Il est plus facile de localiser l'implementation des functions dans le gestionnaire de code JS en survolant le code HTML.
Copy link
Member

Choose a reason for hiding this comment

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

des fonctions


2. Since you don't have to manually attach event listeners in JS, your ViewModel code can be pure logic and DOM-free. This makes it easier to test.
2. Comme vous n'avez pas à attacher manuellement les écoutes dans votre JS, le code du "ViewModel" peut-être purement logique et sans DOM. Ceci rend plus facile les tests.
Copy link
Member

Choose a reason for hiding this comment

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

les écouteurs

Copy link
Member

Choose a reason for hiding this comment

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

« ViewModel »


You might be concerned that this whole event listening approach violates the good old rules about "separation of concerns". Rest assured - since all Vue handler functions and expressions are strictly bound to the ViewModel that's handling the current view, it won't cause any maintenance difficulty. In fact, there are several benefits in using `v-on`:
Vous pouriez être preoccupé que tous ces événements d'écoutes viole la bonne vieille règle de la separation des problèmes.
Rassurez-vous - depuis que le gestionnaire de fonctions et d'expressions est strictement lié à "ViewModel" qui gère la vue courante, cela ne causera aucune difficulté de maintenance. En realité, il y a plusieurs bénéfices à utiliser `v-on` :
Copy link
Member

Choose a reason for hiding this comment

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

Il ne doit pas y avoir de saut de ligne. Le fichier traduit doit faire le même nombre de ligne.

comme le gestionnaire de fonctions et d'expressions est strictement lié au « ViewModel »

« ViewModel »


3. When a ViewModel is destroyed, all event listeners are automatically removed. You don't need to worry about cleaning it up yourself.
3. Quand un "ViewModel est detruit, tout les evenements d'ecoutes sont automatiquenemt retiré". Vous n'aveez pas à vous soucier de le faire vous meme.
Copy link
Member

Choose a reason for hiding this comment

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

« ViewModel »

vous-même

@sylvainpolletvillard
Copy link
Member

@Ti-Bretin Si tu as des suggestions à faire pour améliorer le README pour accueillir les nouveaux, libre à toi d'en faire une pull request :) C'est aussi une bonne manière de contribuer

@MachinisteWeb
Copy link
Member

Hi @Ti-Bretin,

Je pense qu'il serait intéressant que tu corriges déjà les éléments relevés avant que d'autres personne ne refasse une review afin de limiter les éléments qu'ils vont te remonter.

Profitable pour ta correction et pour leur relecture !!

Win - Win ! Mais pas d'Affleulou (http://www.afflelou.com/offre-win-win.html)

@MachinisteWeb
Copy link
Member

MachinisteWeb commented Feb 23, 2017

Dans l'éventualité où @Ti-Bretin se soit fait fauché par un bus, c'est quoi la procédure la plus pratique pour faire les corrections de cette pull request nous-même sans perdre nos remarques ?!

@Ti-Bretin, si tu me lis, sort de sous le bus !

@Ti-Bretin
Copy link
Author

Toutes mes excuses, je m'y recolle aujourd'hui...

@Ti-Bretin
Copy link
Author

Hey guy's, j'ai pas remplacé listener par écouteur car c'est pour moi un contre sens. Un listener en anglais est quelque chose qui reçoit une information. Un écouteur en français courant est un émetteur. Libre à vous de changer pour écouteur mais le terme "écoute" me semble plus comprehensible.

@yann-yinn
Copy link

yann-yinn commented Feb 23, 2017

Hello @Ti-Bretin ! Pour listeners, on traduit partout de la même manière pour être cohérents entre toutes les pages, en l'occurence par écouteurs ( voir cette issue qui référence les choix de traductions qui reviennent souvent : #4 ) .

Par ailleurs en français courant, écouteur n'est pas un émetteur, ( à la rigueur un récepteur (écouteur téléphonique))

@Ti-Bretin
Copy link
Author

Hello, Yann, je comprend la coherence pour toutes les pages, mais si un term est mal traduit il faut savoir modifier, toutes les pages...

Par ailleurs : https://fr.wikipedia.org/wiki/%C3%89couteur
Peut-etre qu'il serait judicieux de demander à plusieurs personnes ce qu'est un écouteur pour eux et ce qu'est une écoute...enfin...pas d'offense, j'essaye juste de faire avancer le shminlblick... ;)

@Ti-Bretin
Copy link
Author

Merci pour les infos @nyl-auster, je persiste à penser que c'est un contre sens mais si c'est la convention...

@MachinisteWeb
Copy link
Member

MachinisteWeb commented Feb 24, 2017

Ça me rappelle cette histoire de « Séparation des préoccupations » pour « Separation of Concerns » Haha ! Le truc qui veut rien dire !!! Mais c'est la convention.

De ton côté envisage le comme ça :

N'entent pas « écouteur » comme un périphérique qui « envoie » du son (genre « met tes écouteurs »), mais comme un homme qui écoute (un écouteur d'histoire (en l'occurrence d’événements)) et donc qui « reçoit ». Pas de contre-sens si on en saisi bien le sens.

C'est exactement la même chose avec « l'observateur » qui « reçoit » comme un homme qui observe.

Dans les deux cas c'est pas un homme, mais c'est le programme bien sur.

En fait si quelqu'un a tord, c'est « l'écouteur (celui qu'on met dans les oreilles) » parce qu'en réalité c'est un « raconteur » puisqu'il produit du son, :D Il faut entendre par la que c'est « l'outil de l'écouteur, celui qui écoute » se périphérique. D'ailleurs en anglais on dit un « speaker » et pas un « listener ».

@yann-yinn
Copy link

yann-yinn commented Feb 24, 2017

je persiste à penser que c'est un contre sens

@Ti-Bretin je comprends ton sentiment mais on doit aussi traduire en fonction de choses objectives telles que rester le plus fidèle au texte original ("listener" = "écouteur" ou "auditeur"), les conventions de traduction déjà existantes ("écouteur") et des sources précises et fiables pour faire des choix en cas de débat sinon nous ne tomberions jamais tous d'accord avec nos sensibilités différentes.

Et sinon même en considérant l'écouteur d'un casque il n'y a pas forcément de contresens : un écouteur dans la production de la chaine sonore n'est pas un émetteur par nature mais un transducteur (il convertit un signal électrique en vibrations de l'air).

C'est dans l'utilisation qu'on en fait tous les jours un système passif qui ne fait rien tant que tu ne le branches pas à une source (un signal électrique entrant).

Copy link

@yann-yinn yann-yinn left a comment

Choose a reason for hiding this comment

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

Merci pour ces premiers correctifs ! on commence à avoir un truc cool, il manque encore quelques corrections suggérées ici et un peu plus haut et on y sera :)

@@ -42,15 +42,15 @@ var example1 = new Vue({
</script>
{% endraw %}

## Method Event Handlers
## Les méthodes de gestion d'évènement"

Choose a reason for hiding this comment

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

un " qui traine à la fin

// you can invoke methods in JavaScript too
example2.greet() // -> 'Hello Vue.js!'
// vous pouvez également invoquer ces méthodes en JavaScript
2.greet() // -> 'Hello Vue.js!'

Choose a reason for hiding this comment

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

example2.greet()

```

``` js
// ...
methods: {
warn: function (message, event) {
avertissement: function (message, event) {

Choose a reason for hiding this comment

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

en effet, remettre "warn"

It is a very common need to call `event.preventDefault()` or `event.stopPropagation()` inside event handlers. Although we can do this easily inside methods, it would be better if the methods can be purely about data logic rather than having to deal with DOM event details.

To address this problem, Vue provides **event modifiers** for `v-on`. Recall that modifiers are directive postfixes denoted by a dot.
C'est un besoin courant que de faire appel ā `event.preventDefault()` ou `event.stopPropagation()` à l'intérieur d'une déclaration de gestionnaire d'évènement. Bien que nous puissions réaliser ceci aisément à l'intérieur de " Methods " , il serait préférable que les méthodes restent purement dediées à la logique des données au lieu d'avoir à prendre en charge les événements du DOM en détails.

Choose a reason for hiding this comment

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

"methods" (pas de majuscule)
"au lieu d'avoir à gérer les détails des évènements du DOM"


To address this problem, Vue provides **event modifiers** for `v-on`. Recall that modifiers are directive postfixes denoted by a dot.
C'est un besoin courant que de faire appel ā `event.preventDefault()` ou `event.stopPropagation()` à l'intérieur d'une déclaration de gestionnaire d'évènement. Bien que nous puissions réaliser ceci aisément à l'intérieur de " Methods " , il serait préférable que les méthodes restent purement dediées à la logique des données au lieu d'avoir à prendre en charge les événements du DOM en détails.
``

Choose a reason for hiding this comment

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

des backsticks qui trainent


You might be concerned that this whole event listening approach violates the good old rules about "separation of concerns". Rest assured - since all Vue handler functions and expressions are strictly bound to the ViewModel that's handling the current view, it won't cause any maintenance difficulty. In fact, there are several benefits in using `v-on`:
Vous pouriez être preoccupé que tous ces événements d'écoutes viole la bonne vieille règle de la séparation des préoccupations.

Choose a reason for hiding this comment

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

Vous pourriez être inquiets du fait que l'ensemble de cette approche d'écouteurs d'évènements (...)


When listening for keyboard events, we often need to check for common key codes. Vue also allows adding key modifiers for `v-on` when listening for key events:
## Modificateurs "key" (Touches)
Lorsque nous écoutons les événements du clavier, nous avons régulièrement besoin de s'assurer des codes des touches. Vue permet également d'ajouter un modificateur de touches pour `v-on`:

Choose a reason for hiding this comment

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

attention, à partir d'ici il y a une ligne de retard : la phrase anglaise commence à la ligne 205


You might be concerned that this whole event listening approach violates the good old rules about "separation of concerns". Rest assured - since all Vue handler functions and expressions are strictly bound to the ViewModel that's handling the current view, it won't cause any maintenance difficulty. In fact, there are several benefits in using `v-on`:
Vous pouriez être preoccupé que tous ces événements d'écoutes viole la bonne vieille règle de la séparation des préoccupations.
Rassurez-vous - depuis que le gestionnaire de fonctions et d'expressions est strictement lié à « ViewModel » qui gère la vue courante, cela ne causera aucune difficulté de maintenance. En realité, il y a plusieurs bénéfices à utiliser `v-on` :

Choose a reason for hiding this comment

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

  • puisque toutes les fonctions et expressions sont strictement liées ...


1. It's easier to locate the handler function implementations within your JS code by simply skimming the HTML template.
1. Il est plus facile de localiser l'implementation des fonctions dans le gestionnaire de code JS en survolant le code HTML.

Choose a reason for hiding this comment

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

implémentation.
les fonctions de gestion ( ou quelque chose approchant)


3. When a ViewModel is destroyed, all event listeners are automatically removed. You don't need to worry about cleaning it up yourself.
3. Quand un « ViewModel » est detruit, tout les evenements d'ecoutes sont automatiquenemt retiré. Vous n'avez pas à vous soucier de le faire vous-même.

Choose a reason for hiding this comment

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

détruit

@MachinisteWeb
Copy link
Member

Pour ma part je ferrais une relecture global après les modifications proposées par @nyl-auster à @Ti-Bretin.

@MachinisteWeb MachinisteWeb changed the title Traduction events.md Review : events.md Mar 4, 2017
@MachinisteWeb MachinisteWeb changed the title Review : events.md events.md Mar 6, 2017
@MachinisteWeb
Copy link
Member

Tu t'en sors @Ti-Bretin ? Plus que ta contribution et components.md et l'utilisation de Vue pour la partie des bases sera couverte !

@MachinisteWeb MachinisteWeb self-requested a review March 14, 2017 09:24
Copy link
Member

@MachinisteWeb MachinisteWeb left a comment

Choose a reason for hiding this comment

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

Merci @Ti-Bretin ! Voici une seconde relecture avec des petites choses que j'ai repéré cette fois. Merci pour tout !

@@ -1,19 +1,19 @@
---
title: Event Handling
title: Gestionnaire des événements
Copy link
Member

Choose a reason for hiding this comment

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

Gestionnaire d'événements

type: guide
order: 9
---

## Listening to Events
## Ecoute des événements
Copy link
Member

Choose a reason for hiding this comment

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

Écoute


For example:
Par exemple:
Copy link
Member

Choose a reason for hiding this comment

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

Par exemple :

(espace devant :)

@@ -25,12 +25,12 @@ var example1 = new Vue({
})
```

Result:
Résultat:
Copy link
Member

Choose a reason for hiding this comment

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

Résultat :

(espace devant :)

@@ -42,15 +42,15 @@ var example1 = new Vue({
</script>
{% endraw %}

## Method Event Handlers
## Les méthodes de gestion d'évènement
Copy link
Member

Choose a reason for hiding this comment

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

d'événements


- `.ctrl`
- `.alt`
- `.shift`
- `.meta`

> Note: On Macintosh keyboards, meta is the command key (⌘). On Windows keyboards, meta is the windows key (⊞). On Sun Microsystems keyboards, meta is marked as a solid diamond (◆). On certain keyboards, specifically MIT and Lisp machine keyboards and successors, such as the Knight keyboard, space-cadet keyboard, meta is labeled “META”. On Symbolics keyboards, meta is labeled “META” or “Meta.
> Note: Sur les claviers Macintosh, meta est la touche commande (⌘). Sur Windows, meta est la touche windows (⊞). Sur les claviers Sun Microsystems, meta est symbolisé par un diamant plein (◆). Sur certains claviers, spécifiquement sur les claviers des machines MIT et Lisp et leurs successeurs, comme le clavier « Knight » et « space-cadet », meta est ecrit « META ». Sur les claviers Symboliques, meta est etiqueté « META » ou « Meta ».
Copy link
Member

Choose a reason for hiding this comment

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

meta est symbolisée


For example:
Par exemple:
Copy link
Member

Choose a reason for hiding this comment

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

Par exemple :

(espace avant :)


You might be concerned that this whole event listening approach violates the good old rules about "separation of concerns". Rest assured - since all Vue handler functions and expressions are strictly bound to the ViewModel that's handling the current view, it won't cause any maintenance difficulty. In fact, there are several benefits in using `v-on`:
Vous pourriez être inquiets du fait que l'ensemble de cette approche d'écouteurs d'évènements viole la bonne vieille règle de la séparation des préoccupations.
Rassurez-vous - puisque toutes les fonctions et expressions sont strictement liées à « ViewModel » qui gère la vue courante, cela ne causera aucune difficulté de maintenance. En realité, il y a plusieurs bénéfices à utiliser `v-on` :
Copy link
Member

Choose a reason for hiding this comment

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

retirer le double espace derrière « VueModel »


1. It's easier to locate the handler function implementations within your JS code by simply skimming the HTML template.
1. Il est plus facile de localiser l'implémentation des fonctions de gestion du code JS en survolant le code HTML.
Copy link
Member

Choose a reason for hiding this comment

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

Je mettrais

code JS en relisant le code HTML


3. When a ViewModel is destroyed, all event listeners are automatically removed. You don't need to worry about cleaning it up yourself.
3. Quand un « ViewModel » est détruit, tout les événements écouteurs sont automatiquenemt retiré. Vous n'avez pas à vous soucier de le faire vous-même.
Copy link
Member

Choose a reason for hiding this comment

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

tous les écouteurs d'événements sont automatiquement retirés.

@MachinisteWeb
Copy link
Member

Encore Merci @Ti-Bretin pour les modifs !

Une fois que tu auras corrigé cette passe (afin de ne pas décaller mes commentaires) il faudra malheureusement faire une màj de ton fork.

Je vois ue ta pull request est maintenant en conflit car le fichier original à changé depuis ta première traduction :(

Il faut que tu fasses un update depuis le fork vuejs-fr/vuejs.org, que tu résolves les conflits et cette pull request devrait être de nouveau modifiable.

Si tu ne sais pas comment faire, les étapes sont les suivantes dans le dossier de ton fork sur ta machine local tu dois ouvrir un invité de commande et

Créer une source de mise à jour

git remote add upstream https://github.com/vuejs-fr/vuejs.org.git

Faire la mise à jour depuis cette source (ici sur le net) vers ta source (ton dossier de fork sur ton pc)

git merge master upstream/master

(en supposant que ta branche soit master)

Modifier les paties en conflit

Ensuite le fichier events.md va comporter les <<<<<< >>>>>> zones de conflits, tu choisis les bonnes que tu retraduis le cas échéant.

Informer que ton conflit est résolue

git add .

Mettre l'info a jour dans ton repository local

Puis tu ajoutes la résolution des conflits à l'index local et tu enregistre ça dans ton repository local

git commit -m "Events.md" conflict resolution.

Mettre l'info a jour dans ton fork en ligne

Finalement tu remonte ça sur ton fork en ligne

git push

C'est tout bon !

Et ça devrait automatiquement marquer ici que la pull request p-e mergé !

@MachinisteWeb
Copy link
Member

@Ti-Bretin, si tu as besoin d'aide pour mettre à jour cette PR tu peux passer sur le chat https://gitter.im/vuejs-fr/vue pour avoir de l'aide en direct.

@MachinisteWeb
Copy link
Member

MachinisteWeb commented Apr 6, 2017

Suite au silence de @Ti-Bretin. Je vais prendre en charge cette PR et faire les ajustements demandé. N'hésite pas à nous faire signe si tu as un peu plus de temps à l'avenir :)

Ce que je vais faire :

  1. Créer une branche Haeresis/vuejs.org:french_events ayant pour upstream source la branche Ti-Bretin/vuejs.org:master.

  2. Faire un fetch de l'upstream vuejs-fr/vuejs.org (qui est à jour vis à vis de vuejs/vuejs.org) et merger la branche master avec ma branche french_events.

  3. Résoudre les conflits et faire les modifications discuté plus haut.

  4. Faire une pull request de Haeresis/vuejs.org:french_events vers vuejs-fr/vuejs.org:master

  5. Fermer cette PR, la référencé dans l'historique des participations et depuis la nouvelle PR et prendre la main pour boucler le travail de @Ti-Bretin.

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