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
add jsx sync modifier #8
base: master
Are you sure you want to change the base?
Conversation
@@ -34,7 +34,8 @@ | |||
"babel-plugin-jsx-event-modifiers": "^2.0.2", | |||
"babel-plugin-jsx-v-model": "^2.0.1", | |||
"babel-plugin-syntax-jsx": "^6.18.0", | |||
"babel-plugin-transform-vue-jsx": "^3.5.0" | |||
"babel-plugin-transform-vue-jsx": "^3.5.0", | |||
"babel-plugin-vue-jsx-sync": "^0.0.4" |
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.
seems it's still too young and lacks of tests?
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.
@egoist yeah, it's young 😂 . I use it in my own project, and it works currently. I can add some unit tests, if you think it is necessary.
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.
We should give it some time to mature first 😄
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.
@egoist OK, Tnx for your time.
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.
@egoist, maybe we can add sync
to event-modifiers, what do you think?
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.
@njleonzhang, if you could make a PR to event-modifiers, I'd be very grateful
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.
@nickmessing, OK, I will try to make one after spring festival
No description provided.