clangd / vscode-clangd

Visual Studio Code extension for clangd
https://marketplace.visualstudio.com/items?itemName=llvm-vs-code-extensions.vscode-clangd
MIT License
592 stars 97 forks source link

"Disable IntelliSense" button should add the setting to disable IntelliSense to Workspace Settings rather than User Settings #595

Open ShiinaKaze opened 3 months ago

ShiinaKaze commented 3 months ago

Please describe the problem.

Conflict with Microsoft C/C++ extension, if disable, it will change vscode user setting. Instead of changing the user Settings, I want to change the workspace Settings. Screenshot 2024-03-16 201008

Logs None

System information Clangd version (from the log, or clangd --version): 17.0.3 clangd extension version: v0.1.26 Operating system: Windows 11

HighCommander4 commented 3 months ago

Instead of changing the user Settings, I want to change the workspace Settings.

You can do that manually, right?

ShiinaKaze commented 3 months ago

Instead of changing the user Settings, I want to change the workspace Settings.

You can do that manually, right?

Sorry, I mean I hope it can be changed, but please forgive me if I am offended

HighCommander4 commented 3 months ago

I'm just trying to understand the request :)

The effect of the "Disable IntelliSense" button currently is to add the setting "C_Cpp.intellisenseEngine": "disabled" to the User Settings.

If you would prefer to have this specified in Workspace Settings, you can currently set this up manually by removing the setting from User Settings and adding it to Workspace Settings.

Are you suggesting that the behaviour of the button be changed to modify Workspace Settings instead (so you don't have to do that manually)?

ShiinaKaze commented 3 months ago

I'm just trying to understand the request :)

The effect of the "Disable IntelliSense" button currently is to add the setting "C_Cpp.intellisenseEngine": "disabled" to the User Settings.

If you would prefer to have this specified in Workspace Settings, you can currently set this up manually by removing the setting from User Settings and adding it to Workspace Settings.

Are you suggesting that the behaviour of the button be changed to modify Workspace Settings instead (so you don't have to do that manually)?

Yes, that's what I mean.

HighCommander4 commented 3 months ago

Thank you for clarifying. I updated the issue title to reflect this.

It would also help to provide some motivation for why you want to have IntelliSense enabled in some projects and not others.

HighCommander4 commented 3 months ago

521 is related