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

fix(runtime-dom): directives trim & number, number is not effect #5839 #5842

Merged
merged 2 commits into from
May 12, 2022

Conversation

liulinboyi
Copy link
Member

close #5839
fix(runtime-dom): directives trim & number, number is not effect #5839

@netlify
Copy link

netlify bot commented May 1, 2022

Deploy Preview for vuejs-coverage ready!

Name Link
🔨 Latest commit b002cee
🔍 Latest deploy log https://app.netlify.com/sites/vuejs-coverage/deploys/62700b290b437000099c2b9c
😎 Deploy Preview https://deploy-preview-5842--vuejs-coverage.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 settings.

@netlify
Copy link

netlify bot commented May 1, 2022

Deploy Preview for vue-sfc-playground ready!

Name Link
🔨 Latest commit b002cee
🔍 Latest deploy log https://app.netlify.com/sites/vue-sfc-playground/deploys/62700b2932c8780008a70dd9
😎 Deploy Preview https://deploy-preview-5842--vue-sfc-playground.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 settings.

@netlify
Copy link

netlify bot commented May 1, 2022

Deploy Preview for vue-next-template-explorer ready!

Name Link
🔨 Latest commit b002cee
🔍 Latest deploy log https://app.netlify.com/sites/vue-next-template-explorer/deploys/62700b29ed55da000812c3ee
😎 Deploy Preview https://deploy-preview-5842--vue-next-template-explorer.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 settings.

@liulinboyi liulinboyi force-pushed the fix-5839 branch 2 times, most recently from d47d2d1 to 14bec25 Compare May 2, 2022 05:16
@hchlq
Copy link
Contributor

hchlq commented May 2, 2022

You are missing the modification to use V-model on the component

@liulinboyi
Copy link
Member Author

You are missing the modification to use V-model on the component

Thanks ! You are right 👍.

@yyx990803 yyx990803 merged commit 71066b5 into vuejs:main May 12, 2022
iwusong pushed a commit to iwusong/core that referenced this pull request May 13, 2022
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.

Trim and number modifiers exist in V-model at the same time, and the number modifier does not take effect
4 participants