vue-google-maps icon indicating copy to clipboard operation
vue-google-maps copied to clipboard

Remove twoWay attributes to all props ?

Open AlexandreBonaventure opened this issue 8 years ago • 2 comments

Hi, first of all thank you for this package, it' good work 👍 . twoWay attributes are here to suggest a good usage for using props. However, it's not required to use props in a twoWay style. I'm using this package along with Vuex, it's generating tons of warnings and it gets a real pain.

Why not removing all twoWay attributes in props definition and let the user decide whether or not to put .sync modifer ? (ex: to draw non-editable marker, etc; ) Moreover, .sync is going to be deprecated.

If the request is approved, I would be glad to submit a PR since I'm already using a custom fork in production.

AlexandreBonaventure avatar Sep 21 '16 17:09 AlexandreBonaventure

I thought the .sync was supposed to be define by both the component and the user, am I wrong ?

GuillaumeLeclerc avatar Sep 23 '16 01:09 GuillaumeLeclerc

Actually, it's not like that. The twoWay attribute intent to clarify prop usage for user, by throwing a warning if it is not used with .sync (it's a hint). In all cases it doesn't allow/prevent prop from being used in twoWay/oneWay style (with .sync) http://vuejs.org/guide/components.html#Prop-Validation.

AlexandreBonaventure avatar Sep 23 '16 18:09 AlexandreBonaventure