-
Notifications
You must be signed in to change notification settings - Fork 115
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
fix: support to show the tag info in the jsDoc #1904
Conversation
96f11a1
to
73f5b2b
Compare
73f5b2b
to
29fd08d
Compare
The Angular VSCode extension will support to show the tags info in the [PR][1]. So the language service can return tags now. [1]: angular/vscode-ng-language-service#1904
The Angular VSCode extension will support showing the tags info in this [PR][1], so the language service can return the tags info now. [1]: angular/vscode-ng-language-service#1904
I would have preferred not needing to do the jsdoc=>markdown conversion in the repo but since I don't have another suggestion, I can't defend blocking this bug fix. Maybe it can be updated at some point in the future. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the fix!
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
https://github.com/microsoft/vscode/blob/18136ed6ec4bfabc58664c8b656839709ea997e8/extensions/typescript-language-features/src/languageFeatures/util/textRendering.ts
https://github.com/microsoft/vscode/blob/18136ed6ec4bfabc58664c8b656839709ea997e8/extensions/typescript-language-features/src/test/unit/textRendering.test.ts
The code is copied from the VSCode extension. The difference is that the code runs in the server in the Angular extension, and runs in the client in the VSCode extension
Fixes #1902
cc @dylhunn