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(completion): add TextEdit for fix #79 #102

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

Conversation

yoyo930021
Copy link
Contributor

@yoyo930021 yoyo930021 commented Dec 12, 2019

The `insertText` is subject to interpretation by the client side.
Some tools might not take the string literally. For example
VS Code when code complete is requested in this example `con<cursor position>`
and a completion item with an `insertText` of `console` is provided itwill only insert `sole`. Therefore it is recommended to use `textEdit` instead
since it avoids additional client side interpretation.
  • Add trace server option

@mergify
Copy link
Contributor

mergify bot commented Mar 22, 2020

This pull request is now in conflicts. Could you fix it?

@mrmlnc
Copy link
Owner

mrmlnc commented Mar 22, 2020

Sorry for trigger from the bot.

I'll try to get back to this PR later. Looks like the original issue is #79 (comment).

@yoyo930021
Copy link
Contributor Author

yoyo930021 commented Mar 22, 2020

I have the same problem when Vue file for only two vscode extensions.
But this solution is too naive, I will update after having time.

@yoyo930021
Copy link
Contributor Author

@mrmlnc
I updated this PR.

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.

Intellisense autocomplete issue in latest build
2 participants