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

Provide a "declaration" icon overlay #113845

Open
elahehrashedi opened this issue Jan 5, 2021 · 6 comments
Open

Provide a "declaration" icon overlay #113845

elahehrashedi opened this issue Jan 5, 2021 · 6 comments
Assignees
Labels
editor-symbols definitions, declarations, references feature-request Request for new features or functionality
Milestone

Comments

@elahehrashedi
Copy link

For C/C++ extension, we would like a "declaration" icon overlay.

the related vscode issue: #23927
here is the cpptools related issue we want to address: microsoft/vscode-cpptools#3117

The result our C/C++ extension wants is just the ability to have our completion/symbol icons match VS:
Completion has a max of 1 modifier:
image

Symbols has a max of 2 modifiers:
image

Looking at C#, they have an additional "internal" icon modifier too (the "protected internal" and "private protected" cases are not given unique icons). Only 1 modifier max.

From a UI design/implementation perspective, it seems like the max icon modifiers would be around 3, with the base icon image rendered in the top left (bigger than the modifiers), and potential modifiers rendered on the bottom left, bottom right, and top right. Unless modifiers could change rendering of the base icon in other ways, such as color modulation or a transparent overlay over the entire base icon (e.g. "strikeout").

@jrieken
Copy link
Member

jrieken commented Jan 6, 2021

/duplicate of #23927

Close this as dupe because #23927 is about the design and candidates for overlay. Please add a comment there with what "modifier" you need

@elahehrashedi
Copy link
Author

@jrieken The conversation on the #23927 issue is locked and limited to collaborators. Would you please unlock it?

@jrieken jrieken reopened this Jan 7, 2021
@jrieken
Copy link
Member

jrieken commented Jan 7, 2021

Oh, sorry. I got that wrong..

@sean-mcmanus
Copy link
Contributor

sean-mcmanus commented May 19, 2021

The original bug report images have some errors. To summarize, with code

class foo
{
public:
	int publicFuncDefinition(){}
	int publicFuncDeclaration();
private:
	int privateFuncDefinition(){}
	int privateFuncDeclaration();
protected:
	int protectedFuncDefinitio(){}
	int protectedFuncDeclaration();
};

we want to be able to provide results like

image

@sean-mcmanus
Copy link
Contributor

sean-mcmanus commented May 20, 2021

This is what our workaround looks like without icons:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor-symbols definitions, declarations, references feature-request Request for new features or functionality
Projects
None yet
Development

No branches or pull requests

4 participants
@jrieken @elahehrashedi @sean-mcmanus and others