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

[suggestion] Summary Comments for Class Access Modifiers #45432

Closed
attrib75 opened this issue Jun 24, 2020 · 1 comment
Closed

[suggestion] Summary Comments for Class Access Modifiers #45432

attrib75 opened this issue Jun 24, 2020 · 1 comment
Labels
Area-IDE Feature Request IDE-IntelliSense Completion, Signature Help, Quick Info Resolution-Won't Fix A real bug, but Triage feels that the issue is not impactful enough to spend time on.
Milestone

Comments

@attrib75
Copy link

It would be useful to have summary comments appear when hovering over access modifiers for a class, to remind us what each access modifier is meant for:

public, private, protected, etc.

Example:
public class MyClass{}

Hover over the "public" access modifier --> tooltip shows "The type or member can be accessed by any other code in the same assembly or another assembly that references it."

@sharwell sharwell added IDE-IntelliSense Completion, Signature Help, Quick Info Feature Request Need Design Review The end user experience design needs to be reviewed and approved. labels Jun 24, 2020
@vatsalyaagrawal vatsalyaagrawal added this to the Backlog milestone Jul 8, 2020
@dibarbet
Copy link
Member

Design meeting notes
We think this feature would be too noisy for experienced developers. There is not enough space in quick info to offer an appropriate explanation and these language features are generally learned relatively quickly. Additionally, F1 help on the modifier will open the relevant documentation where the it can be explained in detail.

I've filed an additional issue where F1 help for combination modifiers is not working correctly.
#46142

@dibarbet dibarbet added the Resolution-Won't Fix A real bug, but Triage feels that the issue is not impactful enough to spend time on. label Jul 20, 2020
@dibarbet dibarbet removed the Need Design Review The end user experience design needs to be reviewed and approved. label Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE Feature Request IDE-IntelliSense Completion, Signature Help, Quick Info Resolution-Won't Fix A real bug, but Triage feels that the issue is not impactful enough to spend time on.
Projects
Archived in project
Development

No branches or pull requests

5 participants