Portion of file where deleted lines show as red

[anuj98]

Type: Bug

In normal file and not in git compare, whenever a line is deleted it shows as red color in the right section. It causes confusion as the this color matches with the color whenver there is an error in code.

VS Code version: Code 1.71.0 (784b017, 2022-09-01T07:36:10.600Z)
OS version: Windows_NT x64 10.0.22000
Modes:
Sandboxed: No

System Info

Item	Value
Screen Reader	no
VM	0%

Extensions (13)

Extension	Author (truncated)	Version
atlascode	atl	2.10.12
emoji-snippets	dev	1.5.3
es7-react-js-snippets	dsz	4.4.3
gitlens	eam	12.2.2
prettier-vscode	esb	9.8.0
auto-close-tag	for	0.5.14
auto-rename-tag	for	0.1.10
vscode-azureappservice	ms-	0.24.3
vscode-azureresourcegroups	ms-	0.5.6
azure-account	ms-	0.11.1
cpptools	ms-	1.12.4
vscode-yaml	red	1.10.1
html5-boilerplate	sid	1.1.1

[vscodenpa]

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.71.2. Please try upgrading to the latest version and checking whether this issue remains.

Happy Coding!

[anuj98]

This is the latest updated version still seeing the same issue

[anuj98]

Which colour are we thinking of here? And what might be the expected ETA?

[lszomoru]

Any thoughts on this @daviddossett, @alexdima?

[daviddossett]

I think it makes sense to use the same color we use for diffs—green for inserted, red for deleted. Not to discount the issue, but I think that benefit outweighs the overlap with errors IMO. It would be strange to use a different color in the editor.

We could possible use a pattern or shape (like a mini version of the line deleted arrow) in the minimap to distinguish it from errors. @lszomoru is it even possible do customize anything beyond the color in the minimap?

[lszomoru]

We could possible use a pattern or shape (like a mini version of the line deleted arrow) in the minimap to distinguish it from errors. @lszomoru is it even possible do customize anything beyond the color in the minimap?

I would have to defer to @alexdima on that.

[alexdima]

Currently only the color can be changed in the minimap gutter decoration rendering

[daviddossett]

I looked into this and it appears that we represent deleted lines in the gutter and errors inline with the line/word(s) highlighted.

I believe this is the right thing to do since it essentially matches what we do in the editor.

Changing the color to be different in the minimap will likely cause more confusion. What did you have in mind, @anuj98?

@alexdima I was going to suggest to override the colors in settings.json but I can't seem to get this to work. Similarly, I can't override the base color editor.errorForeground.

"workbench.colorCustomizations": {
  "editor.minimapGutter.deletedBackground": "#FF53CF",
}

Any ideas why this only works when changing the inherited color value here?

vscode/src/vs/platform/theme/common/colorRegistry.ts

Line 289 in 3747d22

export const editorErrorForeground = registerColor('editorError.foreground', { dark: '#F14C4C', light: '#E51400', hcDark: '#F48771', hcLight: '#B5200D' }, nls.localize('editorError.foreground', 'Foreground color of error squigglies in the editor.'));

[alexdima]

@daviddossett The following worked for me:

"workbench.colorCustomizations": {
	"editorGutter.deletedBackground": "#0000fb",
	"minimapGutter.deletedBackground": "#0000fb",
	"editorError.foreground": "#00ff11",
	"minimap.errorHighlight": "#00ff11",
	"editorOverviewRuler.errorForeground": "#00ff11"
}

[daviddossett]

Ah ok I think my mistake was using "editor.minimapGutter.deletedBackground" instead of "minimapGutter.deletedBackground". Seems to work for me now.

[lszomoru]

@daviddossett, what is the status of this issue? Thank you!

[daviddossett]

I'd suggest we close this issue given the findings above.