Vue-next: #2278 causes kebab case listener not get triggered

Created on 19 Oct 2020  路  16Comments  路  Source: vuejs/vue-next

Version

3.0.1

Reproduction link

https://codesandbox.io/s/dreamy-khayyam-m06w1?file=/src/App.vue

Steps to reproduce

<foo @my-event-name="callEvent" />
this.$emit('my-event-name') // `callEvent` not triggered

What is expected?

The doc recommends using kebab case for event names, but #2278 breaks this.

https://v3.vuejs.org/guide/component-custom-events.html#event-names

What is actually happening?

After https://github.com/vuejs/vue-next/pull/2278 got merged, my code shown above won't work.

(You might want to write some tests before merge. Thanks!)

Most helpful comment

This works correctly in a reproduction using 3.0.1 global build (https://jsfiddle.net/yyx990803/3scy68d5/1/) and also works on a fresh vue-cli project using 3.0.1. There could be a compiler/runtime version mismatch in your project that doesn't properly camelize the event when emitting (make sure your @vue/compiler-sfc is also on 3.0.1).

Closing since it's not reproducible.

Thanks, this was really helpful.

Just updated a project I'm working on to 3.0.2 and ended up here because of events not firing/not being caught - npm update @vue/compiler-sfc fixed that.

All 16 comments

Your reproduction works...

@posva have you clicked the "Hello Vue 3 in CodeSandbox!" ? If it works, the alert should appears.

@posva I don't get any alert from clicking on HelloWorld's body, can't say this works.

Screen Shot 2020-10-19 at 13 04 48

Huh?

Edit: Forced updated the vue version and it doesn't work :+1:

I'm having the same problem.

Since #2278 now all events are camelCased, we now have to camelCase events everywhere, mainly in two places:

  1. when setting emitsOptions on the internal instanc
  2. during the call of emit(), the first argument.

/cc @yyx990803 this is a pretty broad regression from 3.0.0 and might need a hotfix release.

Docs may need an update too.

image

If we make the changes i mentioned, docs could stay as they are. Pascal-Case recommended, but camel case works as well.

This works correctly in a reproduction using 3.0.1 global build (https://jsfiddle.net/yyx990803/3scy68d5/1/) and also works on a fresh vue-cli project using 3.0.1. There could be a compiler/runtime version mismatch in your project that doesn't properly camelize the event when emitting (make sure your @vue/compiler-sfc is also on 3.0.1).

Closing since it's not reproducible.

@LinusBorg @07akioni @posva can you guys confirm it's working on the iife version of vue >=3.0.1 ?

yes, it's working

This works correctly in a reproduction using 3.0.1 global build (https://jsfiddle.net/yyx990803/3scy68d5/1/) and also works on a fresh vue-cli project using 3.0.1. There could be a compiler/runtime version mismatch in your project that doesn't properly camelize the event when emitting (make sure your @vue/compiler-sfc is also on 3.0.1).

Closing since it's not reproducible.

Thanks, this was really helpful.

Just updated a project I'm working on to 3.0.2 and ended up here because of events not firing/not being caught - npm update @vue/compiler-sfc fixed that.

Thanks guys, I don't know why, but it works now automatically...

I'm still getting this issue w/ @vue/compiler-sfc 3.0.4 & vue 3.0.4. Was able to reproduce it in a JSFiddle that only relies on the unpkg dist for vue 3.0.4: https://jsfiddle.net/meo9Lfya/3/
You'll notice that the event works when clicking the "lowercase" button, but not the "kebab" button, but when using "addEventListener" on the element, both work.

Please open a new issue. This will get drowned out.

Was this page helpful?
0 / 5 - 0 ratings