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

JSDoc @constructor shows up in IntelliSense #7243

Closed
egamma opened this issue Feb 25, 2016 · 2 comments · Fixed by #10671
Closed

JSDoc @constructor shows up in IntelliSense #7243

egamma opened this issue Feb 25, 2016 · 2 comments · Fixed by #10671
Assignees
Labels
Bug A bug in TypeScript Fixed A PR has been merged for this issue

Comments

@egamma
Copy link
Member

egamma commented Feb 25, 2016

From @chrisdias on February 24, 2016 22:46

related to #3186?

Testing #3170 and the @ constructor JSDoc comment text shows up in the IntelliSense prompt.

/**
 * Represents a person
 * @constructor
 * @param {string} name The name of the person
 * @param {number} age The age of the person
 */
function Person(name, age) {
    this.name = name;
    this.age = age;
}
var p = new Person

image

image

Copied from original issue: microsoft/vscode#3432

@egamma
Copy link
Member Author

egamma commented Feb 25, 2016

From @joaomoreno on February 25, 2016 8:58

It is up to each language service to parse the comments and provide whatever they want to be rendered in the widget.

@egamma
Copy link
Member Author

egamma commented Feb 25, 2016

Moving to Salsa

@billti billti added this to the TypeScript 2.0 milestone Feb 27, 2016
@billti billti added the Bug A bug in TypeScript label Feb 27, 2016
@mhegazy mhegazy modified the milestones: TypeScript 2.0, TypeScript 2.1 Jun 9, 2016
@mhegazy mhegazy added the Fixed A PR has been merged for this issue label Sep 15, 2016
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A bug in TypeScript Fixed A PR has been merged for this issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants