Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PartialAcceptances auto clear completions
repro in monaco env
TLDR: Issue with partially accepting characters that are already provided in the editor
Reproduction steps:
Expected result:
the completion is persisted on screen as it was partially accepted and the prefix doesn't diverge
Actual result:
The completion is dropped.
Reasoning:
From my investigation this happens due to the explicit signal of cursor position change with type
api
. This is not the case while the user partially accepts completions, with only ghosted text (not yet in the editor).Suggestion:
api
OR update theonCursorChange
to not triggermodel.stop()
when partialAccept is in progress (The earlier proposal might help in understanding what actually changes the position and there do not seem to be existingsource === 'api'
bindings, so no part of the system should be impacted)