Merge pull request microsoft#269682 from microsoft/benibenj/chosen-bug

benibenj · web-flow · commit 3282e87d09fd · 2025-10-03T12:50:35.000+02:00
Fix blue NES arrow display issue for code block highlighting
diff --git a/src/vs/editor/contrib/inlineCompletions/browser/model/inlineCompletionsModel.ts b/src/vs/editor/contrib/inlineCompletions/browser/model/inlineCompletionsModel.ts
@@ -823,6 +823,9 @@ export class InlineCompletionsModel extends Disposable {
 		if (this.showCollapsed.read(reader)) {
 			return false;
 		}
+		if (this._tabShouldIndent.read(reader)) {
+			return false;
+		}
 		if (this._inAcceptFlow.read(reader) && this._appearedInsideViewport.read(reader)) {
 			return true;
 		}
@@ -832,9 +835,6 @@ export class InlineCompletionsModel extends Disposable {
 		if (this._jumpedToId.read(reader) === s.inlineCompletion.semanticId) {
 			return true;
 		}
-		if (this._tabShouldIndent.read(reader)) {
-			return false;
-		}
 
 		return s.cursorAtInlineEdit.read(reader);
 	});

Original file line number	Diff line number	Diff line change
`@@ -823,6 +823,9 @@ export class InlineCompletionsModel extends Disposable {`
`823`	`823`	`if (this.showCollapsed.read(reader)) {`
`824`	`824`	`return false;`
`825`	`825`	`}`
	`826`	`+ if (this._tabShouldIndent.read(reader)) {`
	`827`	`+ return false;`
	`828`	`+ }`
`826`	`829`	`if (this._inAcceptFlow.read(reader) && this._appearedInsideViewport.read(reader)) {`
`827`	`830`	`return true;`
`828`	`831`	`}`
`@@ -832,9 +835,6 @@ export class InlineCompletionsModel extends Disposable {`
`832`	`835`	`if (this._jumpedToId.read(reader) === s.inlineCompletion.semanticId) {`
`833`	`836`	`return true;`
`834`	`837`	`}`
`835`		`- if (this._tabShouldIndent.read(reader)) {`
`836`		`- return false;`
`837`		`- }`
`838`	`838`
`839`	`839`	`return s.cursorAtInlineEdit.read(reader);`
`840`	`840`	`});`