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 overview.md #1863

Closed
wants to merge 2 commits into from
Closed

Update overview.md #1863

wants to merge 2 commits into from

Conversation

Joakal
Copy link

@Joakal Joakal commented Jul 27, 2022

The plugin is NOT recommended for take-over as per original maintainer of Volar vuejs/language-tools#471 (comment)

Description of Problem

This plugin is NOT needed for Vue TS Takeover mode as per original maintainer of Volar

Proposed Solution

Suggest to use the correct plugin only.

Additional Information

The plugin is NOT recommended for take-over as per original maintainer of Volar vuejs/language-tools#471 (comment)
@netlify
Copy link

netlify bot commented Jul 27, 2022

Deploy Preview for vuejs ready!

Name Link
🔨 Latest commit 2be1a2f
🔍 Latest deploy log https://app.netlify.com/sites/vuejs/deploys/62e0ecfc729d300008131aef
😎 Deploy Preview https://deploy-preview-1863--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 settings.

@haoqunjiang
Copy link
Member

Please read on… The next paragraph immediately says that Takeover Mode is a better solution.

We suggest TypeScript Vue Plugin (Volar) by default because it's easier to set up. And the whole paragraph in the documentation is about its mechanism and why it is inferior to Takeover Mode.

Meanwhile, once Takeover Mode is on, TypeScript Vue Plugin (Volar) would be ignored.
So it's better to keep it because it doesn't hurt the performance of Takeover Mode, while still improves the overall experience when the user doesn't take the extra steps to turn on Takeover Mode in the workspace.

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