Adding default array example of correct rule #573

Original file line number	Diff line number	Diff line change
Expand Up		@@ -42,6 +42,10 @@ props: {
		// multiple possible types
		propB: [String, Number],
		// a number with default value
	propC: {
Copy link Member @ota-meshi ota-meshi Sep 14, 2018 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. 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. guillaumebriday reacted with thumbs up emoji Copy link Contributor Author @guillaumebriday guillaumebriday Sep 19, 2018 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. 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 ? Copy link Member @ota-meshi ota-meshi Sep 20, 2018 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. @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 () {}`? guillaumebriday reacted with thumbs up emoji
	type: Array,
	default: () => []
	},
		propD: {
		type: Number,
		default: 100
Expand Down

Navigation Menu