-
Notifications
You must be signed in to change notification settings - Fork 103
Traduction adding-instance-properties.md #22
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
Traduction adding-instance-properties.md #22
Conversation
# Conflicts: # src/api/index.md
# Conflicts: # src/v2/api/index.md
(cherry picked from commit ec5b7c0)
|
||
<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>There may be data/utilities you'd like to use in many components, but you don't want to [pollute the global scope](https://github.com/getify/You-Dont-Know-JS/blob/master/scope%20%26%20closures/ch3.md). In these cases, you can make them available to each Vue instance by defining them on the prototype: | ||
Il peut y avoir des données/utilitaires que vous aimeriez utiliser dans de nombreux composants, mais vous ne voulez pas [polluer le scope global](https://github.com/getify/You-Dont-Know-JS/blob/master/scope%20%26%20closures/ch3.md). Dans ces cas-là, vous pouvez les rendre accessibles dans chaque instance Vue en les définissant dans le prototype: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prototype :
(avec un espace)
|
||
``` js | ||
Vue.prototype.$appName = 'My App' | ||
Vue.prototype.$appName = 'Mon App' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il me semble que dans le guide on a pas traduit le code. Devrions nous le faire ou c'est réservé au Cookbook ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
j'ai toujours traduit les chaînes de caractères dans le code, ce qui est l'ordre du texte, par contre je ne touche pas aux variables et à tout le reste.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je regarderai retro-activement sur ce qui a déjà été traduit
``` | ||
|
||
Now `$appName` is available on all Vue instances, even before creation. If we run: | ||
Maintenant, `$appName` sera accessible dans toutes les instances Vue, même avant leur création. Si nous exécutons: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exécutons :
(avec un espace)
|
||
You may be wondering: | ||
Il se peut que vous vous demandiez: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
demandiez :
|
||
> "Why does `appName` start with `$`? Is that important? What does it do? | ||
> "Pourquoi `appName` commence par un `$`? Est-ce important? Qu'est-ce que ça fait? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
important ?
fait ?
Effectivement dans ce cas la traduction du code est pertinente.
|
||
Now the source of these shared properties is much more obvious: there's an `App` object defined somewhere in the app. To find it, developers need only run a project-wide search. | ||
Maintenant la source de ces propriétés partagées est bien plus évidente: il y a un objet `App` défini quelque-part dans l'application. Pour le trouver, les développeurs ont seulement besoin de rechercher la référence dans le projet. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
évidente :
|
||
Another advantage is that `App` can now be used _anywhere_ in your code, whether it's Vue-related or not. That includes attaching values directly to instance options, rather than having to enter a function to access properties on `this`: | ||
Un autre avantage est que `App` peut maintenant être utilisé _n'importe où_ dans le code, qu'il soit lié à Vue ou non. Cela inclue les valeurs attachées directement aux options des instances, plutôt qu'avoir à entrer dans une fonction pour accéder aux propriétés avec `this`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this
:
@@ -186,8 +186,8 @@ new Vue({ | |||
}) | |||
``` | |||
|
|||
### When Using a Module System | |||
### Quand utiliser un système de modules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Peut-être ouvrir un ticket à la VO mais c'est une question, donc ?
à la fin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ça n'est pas une question selon moi
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alors comme tu l'as dis, on retire le point interrogation précédemment (pour la cohérence)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Soit
Quand faut-il éviter ce pattern ?
Quand utiliser un système de modules ?
ou
Quand faut-il éviter ce pattern
Quand utiliser un système de modules
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est bien une affirmation pour moi.
|
||
``` log | ||
Uncaught TypeError: Cannot read property 'split' of undefined | ||
``` | ||
|
||
## When To Avoid This Pattern |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Peut-être ouvrir un ticket pour informer que c'est une question ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
d'après moi ça n'est pas une question. Je peux retirer le point d'interrogation, bien que cela ait peu d'importance
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Pour moi ce n'est PAS une question, c'est une affirmation dans le sens : nous allons vous dire quand éviter ce pattern
- "pattern" > à traduire ?
|
||
In applications with __no__ module system (e.g. via Webpack or Browserify), there's a pattern that's often used with _any_ JavaScript-enhanced frontend: a global `App` object. | ||
Dans les applications __sans__ systèmes de modules (ex. via Webpack ou Browserify), il y a un pattern souvent utilisé dans _n'importe quel_ front-end amélioré en JavaScript: un objet global `App`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
patron
Là par contre on est pas sur un morceau de code mais sur un objet complet, on peut commencer à ce demander si on ne traduit pas plutôt par patron ?
Tout bon tout ça ! Juste de la ponctuation et un petit questionnement sur le fameux |
C'est dans les tuyaux pour 'Hello ' qui devient 'Hello' vuejs#744 ! |
La ponctuation est corrigée. Pour "pattern" j'ai l'impression qu'on ne tranchera pas facilement entre "motif" "patron" et les autres propositions, alors je pencherai pour garder le mot original pour les raisons évoquées dans #4 |
Pour moi, avant approbation il me reste :
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Voici mes retours sur cette page.
|
||
<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>There may be data/utilities you'd like to use in many components, but you don't want to [pollute the global scope](https://github.com/getify/You-Dont-Know-JS/blob/master/scope%20%26%20closures/ch3.md). In these cases, you can make them available to each Vue instance by defining them on the prototype: | ||
Il peut y avoir des données/utilitaires que vous aimeriez utiliser dans de nombreux composants, mais vous ne voulez pas [polluer le scope global](https://github.com/getify/You-Dont-Know-JS/blob/master/scope%20%26%20closures/ch3.md). Dans ces cas-là, vous pouvez les rendre accessibles dans chaque instance Vue en les définissant dans le prototype : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- scope > en italique
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remplacé par "portée" comme suggéré par @haeresis
// Uh oh - appName is *also* the name of the | ||
// instance property we just defined! | ||
appName: 'The name of some other app' | ||
// Uh oh - appName est *aussi* le nom de la |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "Uh oh" > "Ho ho"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
j'ai mis "Oups" pour ne pas confondre avec le père Noël
|
||
## Real-World Example: Replacing Vue Resource with Axios | ||
## Un exemple en situation réelle : Remplacer Vue Resource avec Axios |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "avec" > "par"
|
||
Let's say you're replacing the [now-retired Vue Resource](https://medium.com/the-vue-point/retiring-vue-resource-871a82880af4). You really enjoyed accessing request methods through `this.$http` and you want to do the same thing with Axios instead. | ||
Disons vous remplacez le [maintenant déprécié Vue Resource](https://medium.com/the-vue-point/retiring-vue-resource-871a82880af4). Vous aimiez vraiment accéder aux méthodes de requête avec `this.$http` et vous voulez faire la même chose avec Axios à la place. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "Disons vous remplacez le [maintenant déprécié Vue Resource]" > "Disons que vous remplaciez [Vue Resource (déprécié)] par ?
- il manque une fin à la phrase pour qu'elle soit cohérente.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L'emploi du passé n'est pas justifié, je reste sur le présent. Comment ça, il manque une fin ? La phrase est cohérente, elle reprend l'originale.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En Français la phrase n'en est pas une.
Je suggère aussi de remplacer "maintenant déprécié" par "(Déprécié)" après le terme.
Il manque le mot "que" après "disons".
|
||
In case you're not aware, methods added to a prototype in JavaScript gain the context of the instance. That means they can use `this` to access data, computed properties, methods, or anything else defined on the instance. | ||
Au cas où vous ne seriez pas au courant, les méthodes ajoutées au prototype en JavaScript obtiennent le contexte de l'instance. Cela signifie qu'elles peuvent utiliser `this` pour accéder aux data, propriétés calculées, méthodes ou toute autre chose définie dans l'instance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "au prototype en JavaScript" > "à prototype en JavaScript" OU "au prototype JavaScript"
- "aux data" > "aux données" OU "aux datas" avec "datas" en italique
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pourquoi "à prototype" ? on ne parle pas de la bibliothèque, c'est bien les méthodes que l'on ajoute au prototype de l'objet. Je ne vois pas pourquoi on devrait mettre "prototype JavaScript" de la même façon.
data est déjà une forme plurielle, c'est le pluriel de datum
|
||
## Alternative Patterns | ||
## Patterns Alternatifs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "patterns" > à traduire ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on s'est mis d'accord sur #4 pour ne pas traduire
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok dans ce cas à mettre en italique. :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En italique donc.
|
||
In applications with __no__ module system (e.g. via Webpack or Browserify), there's a pattern that's often used with _any_ JavaScript-enhanced frontend: a global `App` object. | ||
Dans les applications __sans__ systèmes de modules (ex. via Webpack ou Browserify), il y a un pattern souvent utilisé dans _n'importe quel_ front-end amélioré en JavaScript : un objet global `App`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "patterns" > à traduire ?
- "front-end" > "frontend" + italique
@@ -186,8 +186,8 @@ new Vue({ | |||
}) | |||
``` | |||
|
|||
### When Using a Module System | |||
### Quand utiliser un système de modules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est bien une affirmation pour moi.
|
||
When you have access to a module system, you can easily organize shared code into modules, then `require`/`import` those modules wherever they're needed. This is the epitome of explicitness, because in each file you gain a list of dependencies. You know _exactly_ each one came from. | ||
Quand vous avez accès à un système de modules, vous pouvez facilement organiser le code partagé à travers des modules, puis `require`/`import` ces modules partout où ils sont nécessaires. C'est l'exemple parfait de l'explicité, car chaque fichier obtient alors une liste de dépendances. Vous savez _exactement_ d'où vient chacune d'entre elles. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "C'est l'exemple parfait de l'explicité," > "C'est l'exemple parfait d'une approche explicite,"
- "dépendances" > je ne suis pas sur qu'il faille un pluriel, j'aurais écrit au singulier
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je préfère rester au plus proche de l'original
si c'est une liste de dépendances, c'est qu'il y en a plusieurs donc pluriel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cette partie "C'est l'exemple parfait de l'explicité," ne veut rien dire en français.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je crois que tu as un problème avec le mot "explicité" qui pourtant est bien français: https://fr.wiktionary.org/wiki/explicit%C3%A9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dans ce contexte explicité me semble correctement employé « le fait d'être explicite » (et pas une mauvaise utilisation de « expliquer ».
On peut garder.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Si je ne m'abuse, "explicitness" est un nom commun ici, l'équivalent serait donc en français "l'explicitation" et pas "explicité".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"je dis ça, je dis rien" :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nyl-auster l'explicité est un nom commun aussi. L'explicité est la mesure du caractère explicite tandis que l'explicitation est l'action d'expliciter. Un peu comme capacité et capacitation.
exemple: il a réalisé un gros travail d'explicitation dans son discours afin que ses propos soient d'une grande explicité.
Si vraiment vous tiquez dessus, je vais reformuler pour utiliser l'adjectif explicite mais je vous assure que c'est français, je l'ai lu et entendu à plusieurs reprises.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sylvainpolletvillard ça ne me fait pas tiquer, c'est juste que c'est pas dans mon dictionnaire, ni dans le Larousse en ligne et que je ne l'avais jamais lu. Mais j'en vois une entrée dans le wiktionnaire.
|
||
While certainly more verbose, this approach is definitely the most maintainable, especially when working with other developers and/or building a large app. | ||
Bien que certainement plus verbeux, cette approche est assurément la plus maintenable, particulièrement quand vous travaillez avec d'autres développeurs et/ou construisez une large application. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "cette approche est assurément la plus maintenable" > "cette approche est assurément la plus facile à maintenir"
- "construisez une large application" > "construisez une application complexe"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pour tout ce qui est reformulation, je préfère rester au plus proche de la version originale
|
||
<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>There may be data/utilities you'd like to use in many components, but you don't want to [pollute the global scope](https://github.com/getify/You-Dont-Know-JS/blob/master/scope%20%26%20closures/ch3.md). In these cases, you can make them available to each Vue instance by defining them on the prototype: | ||
Il peut y avoir des données/utilitaires que vous aimeriez utiliser dans de nombreux composants, mais vous ne voulez pas [polluer le scope global](https://github.com/getify/You-Dont-Know-JS/blob/master/scope%20%26%20closures/ch3.md). Dans ces cas-là, vous pouvez les rendre accessibles dans chaque instance Vue en les définissant dans le prototype : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
« instance de Vue » #24 (à discuter)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Polluer la « portée globale » ? Je sais que pour l'objet globale parlé de portée c'est p-e bizarre puisque c'est « tout » mais comme on a déjà traduis plusieurs fois les scopes locaux par « portée locale » ou « portée » tout cours je pense qu'il faut traduire.
Et je reste de ton avis pour ne pas traduire le « scope » de Angular.
|
||
``` js | ||
Vue.prototype.$http = axios | ||
``` | ||
|
||
Then you'll be able to use methods like `this.$http.get` in any Vue instance: | ||
Alors vous serez capables d'utiliser des méthodes comme `this.$http.get` dans n'importe quelle instance Vue : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
« instance de Vue » #24 (à discuter)
@@ -6,7 +6,7 @@ order: 1.1 | |||
|
|||
## Exemple simple | |||
|
|||
Il peut y avoir des données/utilitaires que vous aimeriez utiliser dans de nombreux composants, mais vous ne voulez pas [polluer le scope global](https://github.com/getify/You-Dont-Know-JS/blob/master/scope%20%26%20closures/ch3.md). Dans ces cas-là, vous pouvez les rendre accessibles dans chaque instance Vue en les définissant dans le prototype : | |||
Il peut y avoir des données/utilitaires que vous aimeriez utiliser dans de nombreux composants, mais vous ne voulez pas [polluer la portée globale](https://github.com/getify/You-Dont-Know-JS/blob/master/scope%20%26%20closures/ch3.md). Dans ces cas-là, vous pouvez les rendre accessibles dans chaque instance Vue en les définissant dans le prototype : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instance de Vue
@@ -58,13 +58,13 @@ new Vue({ | |||
}) | |||
``` | |||
|
|||
It would be `"The name of some other app"`, then `"My App"`, because `this.appName` is overwritten ([sort of](https://github.com/getify/You-Dont-Know-JS/blob/master/this%20%26%20object%20prototypes/ch5.md)) by `data` when the instance is created. We scope instance properties with `$` to avoid this. You can even use your own convention if you'd like, such as `$_appName` or `ΩappName`, to prevent even conflicts with plugins or future features. | |||
Cela sera `"Le nom d'une autre app"`, puis `"Mon App"`, car `this.appName` est écrasé ([en quelques sortes](https://github.com/getify/You-Dont-Know-JS/blob/master/this%20%26%20object%20prototypes/ch5.md)) par `data` quand l'instance est créée. Nous limitons la portée des propriétés avec `$` pour éviter ça. Vous pouvez même utiliser votre propre convention si vous préférez, comme `$_appName` ou `ΩappName`, pour en plus prévenir les conflits avec les plugins et les fonctionnalités futures. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
écrasée
car this.appName
est une propriéte.
|
||
Let's say you're replacing the [now-retired Vue Resource](https://medium.com/the-vue-point/retiring-vue-resource-871a82880af4). You really enjoyed accessing request methods through `this.$http` and you want to do the same thing with Axios instead. | ||
Disons vous remplacez le [maintenant déprécié Vue Resource](https://medium.com/the-vue-point/retiring-vue-resource-871a82880af4). Vous aimiez vraiment accéder aux méthodes de requête avec `this.$http` et vous voulez faire la même chose avec Axios à la place. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pour moi également cette phrase ne se suffit pas à elle-même en français. Je propose de fusionner les deux phrases avec une virgule ou plutôt un point virgule ce qui donne un sens à la première tout en dissociant tout de même les deux phrases. Je te laisse voir.
Il y a également Disons qui doit être dissocier du reste de la phrase par une ponctuation ou alors le mot que doit être ajouté car « Disons vous remplacez » est plus quelque chose que l'on dirait à l'oral genre « Disons (petite pause) vous remplacez ».
Une proposition :
Disons que vous remplacez le maintenant déprécié Vue Resource ; vous aimiez vraiment accéder aux méthodes de requête avec
this.$http
et vous voulez faire la même chose avec Axios à la place.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah oui il y a un "que" qui a sauté ici, bien vu
|
||
As long as you're vigilant in scoping prototype properties, using this pattern is quite safe - as in, unlikely to produce bugs. | ||
Tant que vous êtes vigilants à la portée des propriétés du prototype, utiliser ce *pattern* est plutôt sûr - c'est-à-dire, peu probable de produire des bugs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tant que vous êtes vigilant sur la portée
Pour moi on travail « sur » une portée donnée pas « à une portée ».
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
en effet mais ce n'est pas lié à la portée. On dit être vigilant sur quelque-chose ou être vigilant à l'égard de quelque-chose ; ou encore être vigilant quant à quelque-chose
Je viens de refaire une lecture « post » merge et j'ai encore quelque retours à partager. |
bon pour merger ? @haeresis |
Yep ! Et je me laisse pousser la moustache ! |
Ok n'oublie pas de cliquer sur "Approve changes" quand tu valides une review |
Noté. |
cookbook/adding-instance-properties.md
merci pour la review