Adding subheaders to Extension settings #9985
Conversation
There was a problem hiding this comment.
Having "IntelliSense" and "IntelliSense Configuration" seems a little confusing. Maybe rename "IntelliSense" to "IntelliSense Features" or combing them into "IntelliSense"?
Also, "IntelliSense Configuration" seems like it's just for the C_Cpp.default.* settings, in which case maybe calling it "IntelliSense Configuration Defaults" might be better or maybe moved to the IntelliSense section? It seems like an H3 group.
Extension/package.json
Outdated
| "maximum": 65536, | ||
| "scope": "machine" | ||
| }, | ||
| "C_Cpp.files.exclude": { |
There was a problem hiding this comment.
This setting theoretically can apply to all features and not just IntelliSense, so it seems like it should be moved out, e.g. it'll get used by Code Analysis too (and that's in a different section), and we have a bug/TODO tracking making it work for formatting.
|
@Bernardin-MS FYI, my PR at #9979 made changes to the clang-tidy settings, so when you do the merge make sure those changes aren't lost (i.e. that's what's causing the current merge conflict). |
Are these settings System wide? Would something like IntelliSense PC Defaults work? |
How about "Configuration Defaults"? |
The scope can by anything, user to workspace folder. |
|
I worked with @Bernardin-MS on this and we reorganized a few settings/sections. This is the new order: |
|
I'm confused what goes in IntelliSense and what goes in Workspace Configuration. |
Issue: #8237
These are the subheaders
Outdated:
Update:
