Skip to content

Commit c3fe962

Browse files
committed
Open the diff view in current editor, to the side or below.
1 parent 9394467 commit c3fe962

3 files changed

Lines changed: 13 additions & 8 deletions

File tree

extensions/merge-conflict/package.json

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -115,10 +115,15 @@
115115
"description": "%config.autoNavigateNextConflictEnabled%",
116116
"default": false
117117
},
118-
"merge-conflict.openDiffInNewEditor": {
119-
"type": "boolean",
120-
"description": "%config.openDiffInNewEditor%",
121-
"default": false
118+
"merge-conflict.diffViewPosition": {
119+
"type": "string",
120+
"enum": [
121+
"Current",
122+
"Beside",
123+
"Below"
124+
],
125+
"description": "%config.diffViewPosition%",
126+
"default": "Current"
122127
}
123128
}
124129
}

extensions/merge-conflict/package.nls.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,5 @@
1616
"config.autoNavigateNextConflictEnabled": "Whether to automatically navigate to the next merge conflict after resolving a merge conflict.",
1717
"config.codeLensEnabled": "Create a Code Lens for merge conflict blocks within editor.",
1818
"config.decoratorsEnabled": "Create decorators for merge conflict blocks within editor.",
19-
"config.openDiffInNewEditor": "Controls if the diff view should be opened in a new editor group when comparing changes in merge conflicts."
19+
"config.diffViewPosition": "Controls where the diff view should be opened when comparing changes in merge conflicts."
2020
}

extensions/merge-conflict/src/commandHandler.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -125,13 +125,13 @@ export default class CommandHandler implements vscode.Disposable {
125125

126126
const title = localize('compareChangesTitle', '{0}: Current Changes ⟷ Incoming Changes', fileName);
127127
const mergeConflictConfig = vscode.workspace.getConfiguration('merge-conflict');
128-
const openToTheSide = mergeConflictConfig.get<boolean>('openDiffInNewEditor');
128+
const openToTheSide = mergeConflictConfig.get<string>('diffViewPosition');
129129
const opts: vscode.TextDocumentShowOptions = {
130-
viewColumn: vscode.ViewColumn.Active,
130+
viewColumn: openToTheSide === 'Beside' ? vscode.ViewColumn.Beside : vscode.ViewColumn.Active,
131131
selection
132132
};
133133

134-
if (openToTheSide) {
134+
if (openToTheSide === 'Below') {
135135
await vscode.commands.executeCommand('workbench.action.newGroupBelow');
136136
}
137137

0 commit comments

Comments
 (0)