Skip to content

components.md #51

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

Merged
merged 17 commits into from
Apr 11, 2017
Merged

components.md #51

merged 17 commits into from
Apr 11, 2017

Conversation

MachinisteWeb
Copy link
Member

@MachinisteWeb MachinisteWeb commented Apr 7, 2017

PR prête pour la dernière ligne droite.

S'il y a des trucs que j'ai pas modif alors que vous l'aviez demandé c'est que j'en avait discuté dans la PR qui est impossible à lire, n'hésitez pas à re-relever le point.

Traduction complète du fichier `components.md`
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
# Conflicts:
#	src/v2/guide/components.md

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

@sylvainpolletvillard sylvainpolletvillard left a comment

Choose a reason for hiding this comment

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

2ème relecture


``` html
<!-- this passes down a plain string "1" -->
<!-- ceci passe une simple chaîne de caractère "1" -->

Choose a reason for hiding this comment

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

caractères

// object/array defaults should be returned from a
// factory function
// les objets et tableaux par défaut doivent être retournés
// depuis fabrique de fonctions

Choose a reason for hiding this comment

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

retournés par une fabrique de fonctions

@@ -1152,48 +1152,48 @@ Then a `tree-folder-contents` component with this template:
</ul>
```

When you look closely, you'll see that these components will actually be each other's descendent _and_ ancestor in the render tree - a paradox! When registering components globally with `Vue.component`, this paradox is resolved for you automatically. If that's you, you can stop reading here.
En regardant attentivement, vous verrez que ces composants seront en fait l'ancêtre _et_ le descendant l'un de l'autre dans l'arbre de rendu — un paradoxe ! Quand vous inscrivez un composant de manière globale avec `Vue.component`, ce paradoxe est résolu pour vous automatiquement. Si c'est votre cas, vous pouvez arrêter de lire ici.

Choose a reason for hiding this comment

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

il faut choisir entre inscrire et enregistrer comme traduction pour "register", et appliquer la même partout

Copy link
Member Author

Choose a reason for hiding this comment

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

J'ai pris le verbe inscrire partout comme tu l'avais suggérer dans l'autre PR

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
@@ -71,9 +71,9 @@ new Vue({ el: '#example' })
</script>
{% endraw %}

### Enregistrement local
### Inscription local

Choose a reason for hiding this comment

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

locale

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

Merci.

@forresst, tu auras le temps pour une review là dessus ?

Copy link

@forresst forresst left a comment

Choose a reason for hiding this comment

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

C'est bon pour moi

@sylvainpolletvillard sylvainpolletvillard merged commit 2132f96 into vuejs-fr:master Apr 11, 2017
@MachinisteWeb MachinisteWeb deleted the french_components branch April 11, 2017 14:04
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.

3 participants