Skip to content

Commit 0f1adf3

Browse files
authored
only reveal first conflict when not having previous view state (microsoft#154482)
fixes microsoft#153962
1 parent cd8dfa1 commit 0f1adf3

File tree

1 file changed

+11
-10
lines changed

1 file changed

+11
-10
lines changed

src/vs/workbench/contrib/mergeEditor/browser/view/mergeEditor.ts

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -307,16 +307,17 @@ export class MergeEditor extends AbstractTextEditor<IMergeEditorViewState> {
307307
this._ctxBaseResourceScheme.set(model.base.uri.scheme);
308308

309309
const viewState = this.loadEditorViewState(input, context);
310-
this._applyViewState(viewState);
311-
312-
this._sessionDisposables.add(thenIfNotDisposed(model.onInitialized, () => {
313-
const firstConflict = model.modifiedBaseRanges.get().find(r => r.isConflicting);
314-
if (!firstConflict) {
315-
return;
316-
}
317-
318-
this.input1View.editor.revealLineInCenter(firstConflict.input1Range.startLineNumber);
319-
}));
310+
if (viewState) {
311+
this._applyViewState(viewState);
312+
} else {
313+
this._sessionDisposables.add(thenIfNotDisposed(model.onInitialized, () => {
314+
const firstConflict = model.modifiedBaseRanges.get().find(r => r.isConflicting);
315+
if (!firstConflict) {
316+
return;
317+
}
318+
this.input1View.editor.revealLineInCenter(firstConflict.input1Range.startLineNumber);
319+
}));
320+
}
320321

321322

322323
this._sessionDisposables.add(autorunWithStore((reader, store) => {

0 commit comments

Comments
 (0)