Skip to content

Commit 76bfc91

Browse files
authored
Jumping to suggestion should reveal the entire view (microsoft#239205)
* fixes microsoft/vscode-copilot#12422 * 💄
1 parent 2c90783 commit 76bfc91

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

src/vs/editor/contrib/inlineCompletions/browser/model/inlineCompletionsModel.ts

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -768,7 +768,16 @@ export class InlineCompletionsModel extends Disposable {
768768
this._jumpedTo.set(true, tx);
769769
this.dontRefetchSignal.trigger(tx);
770770
this._editor.setPosition(s.inlineEdit.range.getStartPosition(), 'inlineCompletions.jump');
771-
this._editor.revealPosition(s.inlineEdit.range.getStartPosition());
771+
772+
// TODO: consider using view information to reveal it
773+
const isSingleLineChange = s.inlineEdit.range.startLineNumber === s.inlineEdit.range.endLineNumber && !s.inlineEdit.text.includes('\n');
774+
if (isSingleLineChange) {
775+
this._editor.revealPosition(s.inlineEdit.range.getStartPosition());
776+
} else {
777+
const revealRange = new Range(s.inlineEdit.range.startLineNumber - 1, 1, s.inlineEdit.range.endLineNumber + 1, 1);
778+
this._editor.revealRange(revealRange, ScrollType.Immediate);
779+
}
780+
772781
this._editor.focus();
773782
});
774783
}

0 commit comments

Comments
 (0)