-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Enable an insert mode that never replaces (like TypeScript instead of VS) #10613
Comments
It looks likes this would be fixed by switching to returning https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#insertReplaceEdit (instead of the textEdit that is currently returned). |
This feature request is being closed due to insufficient upvotes. Please leave a 👍-upvote or 👎-downvote reaction on the issue to help us prioritize it. When enough upvotes are received, this issue will be eligible for our backlog. |
I think this goes against the spirit of the setting and should be considered a bug, rather than a feature request. Maybe the VS Code devs can chime in to clarify the proper idea of |
@sean-mcmanus Sure, we can consider it a bug, but I'm not sure when we'll get around to fixing it (although the issue is closed, it could still get fixed if we decide to prioritize it later on). |
I'm joining from #11353, is there anything we can do to help prioritize this issue? |
By the way, as there existed previous comments discussing the intent of this settings, and whether it was a bug:
|
@Adnn It's already been prioritized. |
Hi, I've been always finding pretty annoying that either pressing tab/enter the autosuggestion feature always replace the text on the right of the cursor, I always want to "insert". The cpp extension ignores the insertmode setting from vscode and overwrites it. Is it going to be fixed? Thanks! |
Trying to figure out how to fix this annoying behavior. Why is the setting "editor.suggest.insertMode": "insert", ignored? |
Hi, chiming in on this! I personally find it very annoying, and there must be quite a lot of us. How this is not the behavior everyone expects is beyond me, but at least respect the setting. It did use to work at some point, of this I'm certain, so it's not a feature request imho. Thanks in advance! |
Yes, this definitely worked as intended at some point. |
This is actively being worked on and you won't have to wait too much longer. |
Fixed with https://github.com/microsoft/vscode-cpptools/releases/tag/v1.19.0 (pre-release) |
I'm still experiencing the same issue with pre-release 1.19.2. |
@albertcaldas84 I just checked back on this issue, and it works for me with pre-release 1.19.2. To set the value for Cpp, I added the following entry to my
|
If I add this to the settings.json it works. I didn't have any entry, which should assume that "insert" was the default value. |
The default value for |
I apologize in advance for the history lesson, but we chose "replace" even before the VS Code setting existed. This aligned better (though not exactly) with how C++ in Visual Studio behaves. When VS Code introduced the setting they reached out to us and there was a vote about what the default value for the setting would be. We narrowly lost the vote but were given a way to keep our behavior by overriding the default for c and cpp files. We recently suggested a new value "languageDefined" for the insert mode so that we could keep our default and also not require the weird language setting syntax to change it, but it didn't clear the triage bar. |
I would expect the editor with the above settings to never ever ever delete characters, e. g. expect the following outcome:
Originally posted by @Longhanks in #10553 (comment)
The text was updated successfully, but these errors were encountered: