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.

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

Update events.md #2941

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

Conversation

BarnabasSzabolcs
Copy link

Added more explanation to modifiers that require a native event, otherwise generating surprising error messages.

(People get caught in this, eg. mirari/vue-fullscreen#22
When I got caught, I could find many threads online related to trying to use native modifiers without a native event and I didn't find a sound answer other than novices blaming Vue for this and experts not being able to reproduce the problem.)

Note

This repository is for Vue 1.x and 2.x only. Issues and pull requests related to 3.x are managed in the v3 doc repo: https://github.com/vuejs/docs-next.

Added more explanation to modifiers that require a native event, otherwise generating surprising error messages.
@netlify
Copy link

netlify bot commented Apr 11, 2022

Deploy Preview for optimistic-kilby-a8719c ready!

Name Link
🔨 Latest commit 99c84f0
🔍 Latest deploy log https://app.netlify.com/sites/optimistic-kilby-a8719c/deploys/6254b0d4ef8db400081b02ef
😎 Deploy Preview https://deploy-preview-2941--optimistic-kilby-a8719c.netlify.app/v2/guide/events
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

fix: Oops, the event object in $emit needs its event prototype of course, so no spreading notation is possible.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant