Adding default array example of correct rule #573
Conversation
There was a problem hiding this comment.
@guillaumebriday Thank you for this PR!
I have one request.
| @@ -42,6 +42,10 @@ props: { | |||
| // multiple possible types | |||
| propB: [String, Number], | |||
| // a number with default value | |||
| propC: { | |||
There was a problem hiding this comment.
Since it is described in the comment of propE as follows,
// object/array defaults should be returned from a factory function
If you add an example near propE, I think that it is easy to understand.
There was a problem hiding this comment.
Yep you're right, I'll make the change !
Also, I think we coud update the propE to use the arrow function ? What do you think about it ?
There was a problem hiding this comment.
@guillaumebriday Thank you for your suggestion.
I have looked at other documents.
Many seemed to prefer the shorthand method.
(for example here)
Could you write it in shorthand default () {}?
|
Thank you @guillaumebriday 👍 I updated defaults using function shorthand, instead of arrow function. It is a bit more sensible way to write these defaults, methods and any other thing in Vue, as you won't get into trouble when |
|
Hey @michalsnik sorry I could not check it out before. Thanks for the update and the merge ! 👍 |
No description provided.