Skip to content

Update reactivity-utilities.md #1567

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

Closed
wants to merge 1 commit into from
Closed

Conversation

sombriks
Copy link

Description of Problem

small clarification on toRef

Proposed Solution

this small phrase

Additional Information

@netlify
Copy link

netlify bot commented Feb 21, 2022

✔️ Deploy Preview for vuejs ready!

🔨 Explore the source changes: c180866

🔍 Inspect the deploy log: https://app.netlify.com/sites/vuejs/deploys/6212fa82295abc00082a01b8

😎 Browse the preview: https://deploy-preview-1567--vuejs.netlify.app

@skirtles-code
Copy link
Contributor

Thanks for the PR!

I'm afraid I don't understand this change. What do you mean by 'unless the source property is a component property'?

@sombriks
Copy link
Author

Thanks for the PR!

I'm afraid I don't understand this change. What do you mean by 'unless the source property is a component property'?

indeed, it's too short.

since props are readonloy, the resulting ref won't be able to change original prop.

how can i make it clear?

@skirtles-code
Copy link
Contributor

Ah, OK, I understand what you meant.

I've tried to clarify the impact on mutating props in #1624.

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.

2 participants