-
-
Notifications
You must be signed in to change notification settings - Fork 33.7k
.once modifier did'nt perform as expected if my event handler return null #11281
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
Comments
I'm unsure about this one since the condition is explicitly stated at https://github.com/vuejs/vue/blob/dev/src/core/instance/events.js#L36 but I don't know if it's necessary. @gebilaoxiong do you remember why you added the check for |
Thanks @hulkyhawk, let's keep this open as the code could potentially be adapted to allow an event listener to return |
@posva I'm sorry I didn't notice the news; This is not a broken change,The logic used here is the old one: It looks like a signal value, But I can not find the test-case; |
Version
2.6.11
Reproduction link
https://jsfiddle.net/q0v6m8dr/1/
Steps to reproduce
What is expected?
only the first trigger will invoke handler, under '.once' influence
What is actually happening?
subsequent trigger will also invoke handler beside the first one
The text was updated successfully, but these errors were encountered: