search

vuejs / docs

📄 Documentation for Vue 3
https://vuejs.org
Other
2.81k stars 4.1k forks source link

docs: add docs for v-model `update:*` event #2839

Closed xiaodong2008 closed 2 weeks ago

xiaodong2008 commented 2 weeks ago

Added #2810

netlify[bot] commented 2 weeks ago

Deploy Preview for vuejs ready!

Name Link
Latest commit 33e1132e1333487ca087d050417278b34eaee709
Latest deploy log https://app.netlify.com/sites/vuejs/deploys/662f1f92e3a4510008bdfc39
Deploy Preview https://deploy-preview-2839--vuejs.netlify.app
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 configuration.

brc-dd commented 2 weeks ago

Ah, wait this is documented, but it's currently only inside options API block (see later in the document). It should be shown on both composition and options API. It should be similar to what it was earlier - https://github.com/vuejs/docs/blob/7f299d98443efc489115d5f6b64d02094660dbd0/src/guide/components/v-model.md?plain=1 .

xiaodong2008 commented 2 weeks ago

Ah, wait this is documented, but it's currently only inside options API block (see later in the document). It should be shown on both composition and options API. It should be similar to what it was earlier - https://github.com/vuejs/docs/blob/7f299d98443efc489115d5f6b64d02094660dbd0/src/guide/components/v-model.md?plain=1 .

Okay, I will update this.

xiaodong2008 commented 2 weeks ago

Wait, but composition API already have an under the hood part(not complete), and it's not similar to options API, I just completed the part of the parent component, so I think this change is acceptable

https://vuejs.org/guide/components/v-model#under-the-hood