Skip to content

IntelliSense for self.prop = ... class member definitions in JavaScript #16200

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.

Sign up for GitHub

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

Jump to bottom
Closed
mjbvz opened this issue Jun 1, 2017 · 1 comment
Closed

IntelliSense for self.prop = ... class member definitions in JavaScript #16200

mjbvz opened this issue Jun 1, 2017 · 1 comment
Labels
Design Limitation Constraints of the existing architecture prevent this from being fixed VS Code Tracked There is a VS Code equivalent to this issue

Comments

@mjbvz
Copy link
Contributor

mjbvz commented Jun 1, 2017

From microsoft/vscode#27825

TypeScript Version: 2.3.4

Code

var MyClass = function(){
  var self = this;
  self.foo= "";
  this.bar = "";
}

Currently our IntelliSense only suggests bar on instances of MyClass and not foo

I believe this may be a special case of #10868
@mjbvz mjbvz added the VS Code Tracked There is a VS Code equivalent to this issue label Jun 1, 2017
@mjbvz mjbvz mentioned this issue Jun 1, 2017
Closed
@mhegazy
Copy link
Contributor

mhegazy commented Aug 29, 2017

The compiler has no way to track aliasing this for write purposes at the time being.

@mhegazy mhegazy added Salsa Design Limitation Constraints of the existing architecture prevent this from being fixed labels Aug 29, 2017
@mhegazy mhegazy closed this as completed Aug 29, 2017
@microsoft microsoft locked and limited conversation to collaborators Jun 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Design Limitation Constraints of the existing architecture prevent this from being fixed VS Code Tracked There is a VS Code equivalent to this issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mhegazy @mjbvz