Skip to content
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.

Sign up for GitHub

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

Jump to bottom

why v-model:kebab-case not work when used with @update:kebab-case ? #7066

Closed
jizai1125 opened this issue Nov 9, 2022 · 2 comments · Fixed by #7067
Closed

why v-model:kebab-case not work when used with @update:kebab-case ? #7066

jizai1125 opened this issue Nov 9, 2022 · 2 comments · Fixed by #7067
Labels
🔩 p2-edge-case

Comments

@jizai1125
Copy link

jizai1125 commented Nov 9, 2022 •
edited
Loading

Vue version

3.2.41

Link to minimal reproduction

https://sfc.vuejs.org/#eNqVVM2O0zAQfpXBl02l/GwLp5IthXLhDlwwh7SZtqGJbTlOFqnKkRuPAC/HkzC2k/RvxYpKVeT5Zr75PJ6ZI3urVNw2yOYsrTe6UAZqNI1acAFQVEpqA0fQuIUOtlpWcEfOd1xw04PvsZI9Eif2YNmsgwBuNlLUBiqZY/k5K+HBEgXTyWtL7rFG5ZnBE+gx+982YmMKKXqXoJ3A0drhFBO3WdkgRbYuquMiTfwlSD4dDFaqJFd3GUhfRJFjpl8bOVHzrZTROtMkplJFiWAkSPHJ8Q9YH7H0WZ+LeDcERJHP6tMecJ2to01WY3iTW0gDj1If4M/vn+FNHo/8+jEQcpPmtuRXLA+cDWXm7JqEQG8gKPGyiMbq2mQVlqsnZQ2Jn5PEjb/jjSwqxX+oGguW7meLIWgOx+OpfbouTQgcvcY+cG6nThr90mTsABYy37BRlan4Wy0FdbzrJ94DNWfE49+MM2pie+Zsb4yq50nSCHXYxfTqyZKwRDeCJgCjXFbLl/EsfjVN8qI25/YY6ypaa/lYo6aMnIVn5AkZW9SRRpGjRv3PZFe+FwmvsJuk/WB0VIBhOu2sX4xGup8u/IOB0lKBGytX1N7qKjq9rKibsH5llJnY0XPaGpJ9XBs5bguBK5oUKVCYEHZoVo0mreYDDX8mNjislb7iNMdc4HcXT9FZU7rvOUvg3sjqrOfwhbN+6NhXV16sCuPtp2Y7g91uC1xwSAKtN3Rusfi3IfwjPZ9sTBBM4GExNIQnDm5YQ5jNZn5nUZlDmN7f39ujN9gVJ0uMS7kLbq8eTN7EraDmjns9T7lMBi5NyrWAY+dXHd2ms4nGhce6v7Zb9/g=

Steps to reproduce

  <demo v-model:foo-bar="modelVal" @update:foo-bar="update" />

What is expected?

v-model:kebab-case and @update:kebab-case all work.

What is actually happening?

v-model:kebab-case not work，@update:kebab-case work.
but if change to v-model:fooBar, they were all work. I see the compile result，v-model:kebab-case compile to onUpdate:foo-bar, @update:kebab-case compile to onUpdate:fooBar, this should be the reason ?

System Info

No response

Any additional comments?

No response
@sxzz
Copy link
Member

sxzz commented Nov 9, 2022

I just wondering why to open soo many issues. #7026 #7027 #7064 #7065 #7066 😅

@jizai1125
Copy link
Author

jizai1125 commented Nov 9, 2022 •
edited
Loading

I just wondering why to open soo many issues. #7026 #7027 #7064 #7065 #7066 😅

sorry 😅 it was careless

@sxzz sxzz mentioned this issue Nov 9, 2022
Merged
yyx990803 pushed a commit that referenced this issue Nov 14, 2022
@sxzz
fix(v-model): ensure v-model listener casing is consistent with manua…
87c72ae 
…l v-on listener (#7067)

fix #7066
chrislone pushed a commit to chrislone/core that referenced this issue Feb 4, 2023
@sxzz
fix(v-model): ensure v-model listener casing is consistent with manua…
74f2025 
…l v-on listener (vuejs#7067)

fix vuejs#7066
zhangzhonghe pushed a commit to zhangzhonghe/core that referenced this issue Apr 12, 2023
@sxzz @zhangzhonghe
fix(v-model): ensure v-model listener casing is consistent with manua…
7a9efc6 
…l v-on listener (vuejs#7067)

fix vuejs#7066
@github-actions github-actions bot locked and limited conversation to collaborators Sep 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🔩 p2-edge-case
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(compiler-core): case casting of v-model
2 participants
@sxzz @jizai1125