Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Prop modifier for v-model #140

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

CyberAP
Copy link
Contributor

@CyberAP CyberAP commented Mar 5, 2020

Add a prop modifier for v-model to simplify wrapper components.

Rendered

@backbone87
Copy link

Ref #21

@CyberAP
Copy link
Contributor Author

CyberAP commented Mar 5, 2020

Ref #21

Thanks for linking a related issue. I knew someone has already proposed that!

But there are some key differences from the #21 proposal that I'd like to mention:

@victornpb
Copy link

I really like this, wrapping something should be just as easy as "wrapping it"!

@yooneskh
Copy link

I absolutely support this and this is the best among rfcs about this problem. This really reduces a lot of code when we wrap things.

v-model is already (presented as) a syntactic sugar for binding and emitting, so a modifier for v-model would modify the syntactic sugar a bit.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants