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

Relecture de forms.md #16

Merged
merged 3 commits into from
Jun 23, 2017
Merged

Relecture de forms.md #16

merged 3 commits into from
Jun 23, 2017

Conversation

MachinisteWeb
Copy link
Member

L'un des dernier fichier avant d'avoir la traduction de la doc Vuex complètement monitoré :)

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
Copy link
Member

@Kocal Kocal left a comment

Choose a reason for hiding this comment

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

Une petite erreur

docs/en/forms.md Outdated

The "Vuex way" to deal with it is binding the `<input>`'s value and call an action on the `input` or `change` event:
La Vuex peut gérer ça en liant la valeur de l'`input` et en appelant une action sur l'évènement `input` ou `change` :
Copy link
Member

Choose a reason for hiding this comment

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

La « façon Vuex » pour gérer cela est de lier

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
@MachinisteWeb
Copy link
Member Author

Merci @Kocal ! J'ai utilisé « la méthode Vuex » en référence à « la méthode ... » couramment utilisé en français. « La façon X » ne me semble pas quelque chose de naturellement français.

@Kocal
Copy link
Member

Kocal commented Jun 22, 2017

Oui c'est pas faux, peut-être que je traduis « ...('s) way » trop littéralement. 🤔

Je merge + close + delete ?

@MachinisteWeb
Copy link
Member Author

MachinisteWeb commented Jun 22, 2017

Oui tu peux, surtout valide la PR :)

@Kocal Kocal merged commit 7638407 into working Jun 23, 2017
@Kocal Kocal deleted the form branch June 23, 2017 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants