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

Loose IntelliSense for angularjs updating from 0.10.9 #8163

Closed
miguelchico opened this issue Jun 26, 2016 · 10 comments
Closed

Loose IntelliSense for angularjs updating from 0.10.9 #8163

miguelchico opened this issue Jun 26, 2016 · 10 comments
Assignees
Labels
javascript JavaScript support issues

Comments

@miguelchico
Copy link

  • VSCode Version: 1.2.1
  • OS Version: Windows 7

Steps to Reproduce:
I´m moving from version 0.10.9 to 1.2.1 and I´ve lost IntelliSense information

  1. Create a project
  2. Create an angular 1 controller like this:
    image
  3. Create a jsconfig.json like this:

image
4. Using typings install angular TypeScript definition

In version 0.10.9 IntelliSense was working in all places I needed in this controller.
It was working in angular:
image

And it also was working in injected dependencies:
image

From version 0.10.11 the IntelliSense information for the same file and same project is different.
It still work in angular but it doesn't for injected dependencies anymore:
image

I've tried with versions 0.10.11 and 1.2.1 and it doesn't work.
I've checked the release notes from 0.10.9 to 1.10.11 and the main difference was the inclusion of Salsa, but I couldn't find any change I have to do in my project to keep it working.

Any suggestion?

@dbaeumer dbaeumer added the javascript JavaScript support issues label Jun 27, 2016
@dbaeumer
Copy link
Member

@egamma assigning to you since you provided quite some support for people switching over to Salsa.

@egamma
Copy link
Member

egamma commented Jun 29, 2016

@miguelchico we are sorry, but support for intellisense for injected dependencies has been lost as part of the migration to the new Salsa implementation.

@egamma
Copy link
Member

egamma commented Jun 29, 2016

@miguelchico did you try to type the injected dependencies using JS Doc comments?
https://code.visualstudio.com/docs/languages/javascript#_intellisense

@miguelchico
Copy link
Author

@egamma that's a pitty....

Yes, I tried using JS Doc comments and it works, but now I have to write a lot of types manually in my comments and maintain them.

There is no play to get back the support?

Thank you.

@egamma
Copy link
Member

egamma commented Jul 3, 2016

@miguelchico

There is no play to get back the support?

To be honest, we did something very 'pragmatic'/hackish to enable this in the past. pls file a feature request against the TypeScript project https://github.com/Microsoft/TypeScript/issues/new

@bmontgom
Copy link

@miguelchico Did you ever file a feature request? I'm wanting this feature as well.

@miguelchico
Copy link
Author

I've done right know.

microsoft/TypeScript#9600

@egamma, could you please check if it's what you needed?

@bmontgom
Copy link

Thanks, @miguelchico.
@egamma, is this something that will be difficult for the VSCode team to do, and as such should I expect this to take a long time (more than a month) to become available again?

@egamma
Copy link
Member

egamma commented Jul 13, 2016

@miguelchico issue LGTM, thanks.

@bmontgom the JS language service is done by the TS team, if they do it (will talk to them), then it would be in a future TypeScript release.

Closing this issue, since it is now tracked in TypeScript repository.

@egamma egamma closed this as completed Jul 13, 2016
@egamma
Copy link
Member

egamma commented Jul 13, 2016

@bmontgom @miguelchico one immediate workaround would be to augment one of the JSDoc comment generator extensions and make it angular aware so that it generates the JSDoc comments.

https://marketplace.visualstudio.com/items?itemName=stevencl.addDocComments
https://marketplace.visualstudio.com/items?itemName=joelday.docthis

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
javascript JavaScript support issues
Projects
None yet
Development

No branches or pull requests

4 participants