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.

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

Classify items in quick info #31

Closed
GeorgeAlexandria opened this issue Aug 21, 2018 · 1 comment · Fixed by #52
Closed

Classify items in quick info #31

GeorgeAlexandria opened this issue Aug 21, 2018 · 1 comment · Fixed by #52
Assignees
@GeorgeAlexandria
Labels
discussion A issue need to be discussed before take or deprecate it features help wanted investigation A issue must be investigated to find it cause and solution solved A issue was solved
Milestone
v3.0.0

Comments

@GeorgeAlexandria
Copy link
Owner

Would be nice if a text in the quick info will classify as in the text editor in the first approximate
@GeorgeAlexandria GeorgeAlexandria added features investigation A issue must be investigated to find it cause and solution discussion A issue need to be discussed before take or deprecate it labels Aug 21, 2018
@GeorgeAlexandria
Copy link
Owner Author

After a lot of investigation I understand that doesn't exist a way to integrate a custom classifier to the default generated text for quick info. Is only one way to apply custom classifications to quick info – create text of quick info by hand and apply the relevant ClassifiedTextRun of analyzing node.

Besides, in the Microsoft.CodeAnalysis.*.2.9.0 will introduced a public QuickInfoService that allows to get a raw tagged text for applying to them a some custom analysis, but unfortunatly, the WellKnownTags doesn't contain a couple of CoCo classifications. So, for full supporting CoCo classifications in the quick info need to generate a quick info by hand anyway.

Well, I will try to generate full quick info text for nodes and symbols by myself as close as possible to the default quick info text, but it will be not so far.

Would be grateful If I really miss something if someone from community will have a some suggestions for issue.

@GeorgeAlexandria GeorgeAlexandria self-assigned this Dec 12, 2018
@GeorgeAlexandria GeorgeAlexandria added this to the v3.0.0 milestone Jan 27, 2019
@GeorgeAlexandria GeorgeAlexandria mentioned this issue Mar 19, 2019
Merged
@GeorgeAlexandria GeorgeAlexandria added the solved A issue was solved label Mar 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GeorgeAlexandria GeorgeAlexandria

Labels
discussion A issue need to be discussed before take or deprecate it features help wanted investigation A issue must be investigated to find it cause and solution solved A issue was solved
Projects
None yet
Milestone
v3.0.0
Development

Successfully merging a pull request may close this issue.

Classified items in quick info GeorgeAlexandria/CoCo
1 participant
@GeorgeAlexandria