untitled - do not compute label when first line change is >50 column

Benjamin Pasero · Benjamin Pasero · commit 2e92c8fb4a1e · 2020-01-29T08:23:47.000+01:00
diff --git a/src/vs/workbench/common/editor/untitledTextEditorModel.ts b/src/vs/workbench/common/editor/untitledTextEditorModel.ts
@@ -226,8 +226,8 @@ export class UntitledTextEditorModel extends BaseTextEditorModel implements IUnt
 			this.setDirty(true);
 		}
 
-		// Check for name change if first line changed
-		if (e.changes.some(change => change.range.startLineNumber === 1 || change.range.endLineNumber === 1)) {
+		// Check for name change if first line changed in the range of 0-50 columns
+		if (e.changes.some(change => (change.range.startLineNumber === 1 || change.range.endLineNumber === 1) && change.range.startColumn <= UntitledTextEditorModel.FIRST_LINE_NAME_MAX_LENGTH)) {
 			this.updateNameFromFirstLine();
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -226,8 +226,8 @@ export class UntitledTextEditorModel extends BaseTextEditorModel implements IUnt`
`226`	`226`	`this.setDirty(true);`
`227`	`227`	`}`
`228`	`228`
`229`		`- // Check for name change if first line changed`
`230`		`- if (e.changes.some(change => change.range.startLineNumber === 1 \|\| change.range.endLineNumber === 1)) {`
	`229`	`+ // Check for name change if first line changed in the range of 0-50 columns`
	`230`	`+ if (e.changes.some(change => (change.range.startLineNumber === 1 \|\| change.range.endLineNumber === 1) && change.range.startColumn <= UntitledTextEditorModel.FIRST_LINE_NAME_MAX_LENGTH)) {`
`231`	`231`	`this.updateNameFromFirstLine();`
`232`	`232`	`}`
`233`	`233`