Skip to content

Commit 21aa6be

Browse files
authored
Update Merge Editor Colors (microsoft#159199)
Revert merge editor color changes
1 parent 22c16c4 commit 21aa6be

File tree

2 files changed

+12
-27
lines changed

2 files changed

+12
-27
lines changed

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

Lines changed: 6 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -8,55 +8,37 @@ import { registerColor } from 'vs/platform/theme/common/colorRegistry';
88

99
export const diff = registerColor(
1010
'mergeEditor.change.background',
11-
{ dark: '#40A6FF33', light: '#40A6FF33', hcDark: '#40A6FF33', hcLight: '#40A6FF33', },
11+
{ dark: '#9bb95533', light: '#9bb95533', hcDark: '#9bb95533', hcLight: '#9bb95533', },
1212
localize('mergeEditor.change.background', 'The background color for changes.')
1313
);
1414

1515
export const diffWord = registerColor(
1616
'mergeEditor.change.word.background',
17-
{ dark: '#9bb9551e', light: '#9bb9551e', hcDark: null, hcLight: null, },
17+
{ dark: '#9bb9551e', light: '#9bb9551e', hcDark: '#9bb9551e', hcLight: '#9bb9551e', },
1818
localize('mergeEditor.change.word.background', 'The background color for word changes.')
1919
);
2020

2121
export const conflictBorderUnhandledUnfocused = registerColor(
2222
'mergeEditor.conflict.unhandledUnfocused.border',
23-
{ dark: '#ffa6007a', light: '#ffa6007a', hcDark: '#ffa60080', hcLight: '#ffa60080', },
23+
{ dark: '#ffa6007a', light: '#ffa6007a', hcDark: '#ffa6007a', hcLight: '#ffa6007a', },
2424
localize('mergeEditor.conflict.unhandledUnfocused.border', 'The border color of unhandled unfocused conflicts.')
2525
);
2626

27-
export const conflictBackgroundUnhandledUnfocused = registerColor(
28-
'mergeEditor.conflict.unhandledUnfocused.background',
29-
{ dark: '#ffa60026', light: '#ffa60080', hcDark: '#ffa60026', hcLight: '#ffa6007a', },
30-
localize('mergeEditor.conflict.unhandledUnfocused.background', 'The background color of unhandled unfocused conflicts.')
31-
);
32-
3327
export const conflictBorderUnhandledFocused = registerColor(
3428
'mergeEditor.conflict.unhandledFocused.border',
35-
{ dark: '#ffa600BF', light: '#ffa600', hcDark: '#ffa600', hcLight: '#ffa600', },
29+
{ dark: '#ffa600', light: '#ffa600', hcDark: '#ffa600', hcLight: '#ffa600', },
3630
localize('mergeEditor.conflict.unhandledFocused.border', 'The border color of unhandled focused conflicts.')
3731
);
3832

3933
export const conflictBorderHandledUnfocused = registerColor(
4034
'mergeEditor.conflict.handledUnfocused.border',
41-
{ dark: '#40C8AE49', light: '#40C8AE49', hcDark: '#40C8AE', hcLight: '#40C8AE', },
35+
{ dark: '#86868649', light: '#86868649', hcDark: '#86868649', hcLight: '#86868649', },
4236
localize('mergeEditor.conflict.handledUnfocused.border', 'The border color of handled unfocused conflicts.')
4337
);
4438

45-
export const conflictBackgroundHandledUnfocused = registerColor(
46-
'mergeEditor.conflict.handledUnfocused.background',
47-
{ dark: '#40C8AE33', light: '#40C8AE33', hcDark: '#40C8AE33', hcLight: '#40C8AE33', },
48-
localize('mergeEditor.conflict.handledUnfocused.background', 'The background color of handled unfocused conflicts.')
49-
);
50-
51-
export const conflictBackgroundHandledFocused = registerColor(
52-
'mergeEditor.conflict.handledFocused.background',
53-
{ dark: '#40C8AE33', light: '#40C8AE33', hcDark: '#40C8AE33', hcLight: '#40C8AE33', },
54-
localize('mergeEditor.conflict.handledFocused.background', 'The background color of handled focused conflicts.')
55-
);
56-
5739
export const conflictBorderHandledFocused = registerColor(
5840
'mergeEditor.conflict.handledFocused.border',
59-
{ dark: '#40C8AEBF', light: '#40C8AEBF', hcDark: '#40C8AE', hcLight: '#40C8AE', },
41+
{ dark: '#c1c1c1cc', light: '#c1c1c1cc', hcDark: '#c1c1c1cc', hcLight: '#c1c1c1cc', },
6042
localize('mergeEditor.conflict.handledFocused.border', 'The border color of handled focused conflicts.')
6143
);
6244

src/vs/workbench/contrib/mergeEditor/browser/view/media/mergeEditor.css

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,10 @@
5858
flex-grow: 0;
5959
}
6060

61+
.merge-editor-diff {
62+
background-color: var(--vscode-mergeEditor-change-background);
63+
}
64+
6165
.merge-editor-diff-word {
6266
background-color: var(--vscode-mergeEditor-change-word-background);
6367
}
@@ -176,9 +180,8 @@
176180
}
177181

178182
.conflict-zone-root {
183+
background-color: var(--vscode-mergeEditor-change-background);
179184
border: 1px dashed var(--vscode-mergeEditor-conflict-unhandledUnfocused-border);
180-
background-color: var(--vscode-mergeEditor-conflict-unhandledUnfocused-background);
181-
182185
height: 90%;
183186
display: flex;
184187
align-items: center;
@@ -195,11 +198,11 @@
195198
}
196199

197200
.conflict-zone-root .text {
201+
background: var(--vscode-mergeEditor-conflictingLines-background);
198202
margin-left: auto;
199203
padding: 0 8px;
200204
display: flex;
201205
align-items: center;
202-
/* background: var(--vscode-mergeEditor-conflict-unhandledUnfocused-background); */
203206
height: 100%;
204207
white-space: nowrap;
205208
overflow: hidden;

0 commit comments

Comments
 (0)