night-owl-vscode-theme icon indicating copy to clipboard operation
night-owl-vscode-theme copied to clipboard

diffeditor borders are visually disruptive

Open xmatthias opened this issue 4 years ago • 2 comments

Screenshots the (quite bright) borders between the lines make it quite difficult to concentrate on the diff lines itself, when it's a diff of more than 2 lines.

image

The borders themselfs are caused by diffEditor.insertedTextBorder and diffEditor.removedTextBorder - however setting them to transparent (or any other color) will only make them stand out less, not remove them completely (i suspect it's something in vscode that causes this overlap if border-properties are set).

The only way i found to get rid of them is to edit the original file (in my case ~/.vscode/extensions/sdras.night-owl-2.0.1/themes/Night\ Owl-color-theme.json), and remove these attributes from the file (colorCustomizations don't seem to allow unsetting these properties).

The result will look as follows - which i think is a lot less intrusive, and makes the text/diff stand out a lot more.

image

If desired, i can make a PR removing this (i feel this small change will make this Theme perfect).

xmatthias avatar Jan 22 '22 09:01 xmatthias

Thanks, I was also wondering how to fix this, as the thick lines in the diff editor are quite annoying and ugly. I hope this change makes it into the theme, as it is much better like this.

alxgnon avatar Jan 26 '22 21:01 alxgnon

You should open a PR for this! I stopped using this theme because of this issue. Thanks for figuring it out.

brianmoran avatar Mar 04 '22 21:03 brianmoran