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

[Take Over mode] Not inserting * on enter in JSDoc #974

Closed
zardoy opened this issue Feb 21, 2022 · 5 comments
Closed

[Take Over mode] Not inserting * on enter in JSDoc #974

zardoy opened this issue Feb 21, 2022 · 5 comments
Labels

Comments

@zardoy
Copy link
Contributor

zardoy commented Feb 21, 2022

/**
 * |
 */
let interval: number

TS: Pressing enter in | position inserts asterisk in new line.
This is not a case when take over mode is enabled.

@zardoy
Copy link
Contributor Author

zardoy commented Feb 21, 2022

I'm just curios. How it compares to builtin TypeScript plugin?

I can see that files are not being downloaded / patched from vscode repo. I also wonder why original TS plugin isn't being patched by using something like monkey patch.

If the implementation is independent, is it possible to expect some extra features that we have been waiting for years, such as correct sorting of suggestions or disabling import suggestions from specific library?

EDIT: Just got some thoughts from #471. However "hacking" into ext and ext reload after each update (that occurs once a month) wasn't so bad solution IMO.

@johnsoncodehk
Copy link
Member

johnsoncodehk commented Mar 2, 2022

Can you reproduce this problem in https://github.com/johnsoncodehk/volar-starter?

If the implementation is independent, is it possible to expect some extra features that we have been waiting for years, such as correct sorting of suggestions or disabling import suggestions from specific library?

We need to be consistent with VSCode, because their designed behavior is verified by more users, and changing it in volar might be counterproductive.

EDIT: Just got some thoughts from #471. However "hacking" into ext and ext reload after each update (that occurs once a month) wasn't so bad solution IMO.

Svelte is also looking a way to avoid it, I think we will not go back. 😅
sveltejs/language-tools#1358

@zardoy
Copy link
Contributor Author

zardoy commented Mar 2, 2022

Can you reproduce this problem in johnsoncodehk/volar-starter?

I think it'd more useful if I just attach a demo:

Mar-02-2022 11-25-04

As you can see pressing enter actually inserts asterisk. But takeover (when builtin TypeScript features ext disabled) mode doesn't do this.

@zardoy
Copy link
Contributor Author

zardoy commented Mar 6, 2022

Can't repro anymore, everything works perfectly. Have no idea why it didn't work before.

@zardoy zardoy closed this as completed Mar 6, 2022
@johnsoncodehk
Copy link
Member

I think it was changed in microsoft/vscode#125748.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants