Support white-space changes in the gutter change markers

[eldarshamukhamedov]

• VSCode Version: 1.3.1 (e6b4afa)
• OS Version: Linux Mint 18 (Linux 4.4.0-31-generic x86_64 GNU/Linux)
• Git Version: 2.9.2

Steps to Reproduce:

1. Init new git repo: git init test && cd test
2. Create a test file and commit it: echo "Hello" > README.md && git add . && git commit -m "test"
3. Prepend some spaces: echo " Hello" > README.md
4. Open in VSCode: code .
5. Note that README.md is correctly tracked as modified in the Git/VCS tab
6. Open README.md
7. Note that the white-space change does not show up in the gutter (.margin-view-overlays)

In this screenshot, I would expect a blue glyph (.git-dirty-modified-diff-glyph) to show up on line 1:

My user settings:

{
    "editor.renderWhitespace": true,
    "editor.tabSize": 2,
    "explorer.openEditors.visible": 0,
    "editor.rulers": [100, 120]
}

[rmunn]

I would actually prefer for whitespace changes not to show up in the Git gutter, at least in some circumstances. So I would like a configurable option to specify whether whitespace changes should or should not be considered in the gutter.

[eldarshamukhamedov]

@rmunn Agreed, more config options is always better. I'm coming from Atom/WebStorm, so this behavior threw me off a bit.

[joaomoreno]

@alexandrudima What do you think about providing a checkbox to enable/disable whitespace in the diff editor?

[alexdima]

@joaomoreno 👍

[Dominik-K]

@joaomoreno It would be nice to whitelist the activation for specific file formats, as well. E.g. in YAML or Python files, indentation changes code behavior.

[Gruntfuggly]

If vscode is going to ignore whitespace changes in the diff view, it should not stage the changes either.

[antcodd]

There is a setting for this for the diff editor that seems to work, diffEditor.ignoreTrimWhitespace. Why not the gutter?

[sant123]

@antcodd using this setting worked.

{
  "diffEditor.ignoreTrimWhitespace": false
}

Thank you!

[eldarshamukhamedov]

Thanks, @sant123, that works for the Git diff view and is very helpful :) Git gutter still ignores trimmed whitespace, though, so I think keeping this issue open is still valid.

[vasylnakvasiuk]

+1
Please add similar to:
"gitGutter.ignoreTrimWhitespace": true/false

[vkbo]

Although it is fine to be able to turn this off, the gutter really should he able to show the changes that actually goes into the git commit. I depend on this feature for a couple of the projects I work on as unnecessary changes in indentation clutters the change history.

[joaomoreno]

Actually, the diffEditor.ignoreTrimWhitespace is already working for gutter decorations: #40411

[vkbo]

Yes, but this is not about the diff editor, but the editor. This issue is not resolved as of 1.19.2.