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

Suggest widget fails to add import #116765

Closed
joaomoreno opened this issue Feb 16, 2021 · 6 comments
Closed

Suggest widget fails to add import #116765

joaomoreno opened this issue Feb 16, 2021 · 6 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) suggest IntelliSense, Auto Complete typescript Typescript support issues

Comments

@joaomoreno
Copy link
Member

joaomoreno commented Feb 16, 2021

I hit this soooo often, and for soooo long, I am surprised no one else sees it:

  1. In a TS file, type a symbol from another file
  2. At the end, press Ctrl Space and select it with Enter to add it to the imports

🐛 TypeScript doesn't add any import statement.

I have a very hard time reproducing this, unfortunately. As I tried to repro for a recording I completely failed. @mjbvz Can you review the code here and see if there are any holes in the logic which might cause this?

cc @jrieken

@joaomoreno
Copy link
Member Author

Yup

recording (14)

@mjbvz
Copy link
Collaborator

mjbvz commented Feb 16, 2021

Is this in the latest insiders?

@joaomoreno
Copy link
Member Author

Yeah, tho I hit this for at least 2+ months.

@mjbvz
Copy link
Collaborator

mjbvz commented Feb 17, 2021

I should have mostly fixed this this iteration, which is likely why you are having trouble reproducing it

Can you share a specific repo where you still see it?

@joaomoreno
Copy link
Member Author

I only ever saw this in microsoft/vscode. I can never repro on demand unfortunately...

@mjbvz
Copy link
Collaborator

mjbvz commented Feb 17, 2021

Ok, I'm going to close this as a duplicate of #109439 then

If you find a specific case where this doesn't work, please let me know

@mjbvz mjbvz closed this as completed Feb 17, 2021
@mjbvz mjbvz added *duplicate Issue identified as a duplicate of another issue(s) suggest IntelliSense, Auto Complete typescript Typescript support issues labels Feb 17, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Apr 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) suggest IntelliSense, Auto Complete typescript Typescript support issues
Projects
None yet
Development

No branches or pull requests

2 participants