vscode-yaml icon indicating copy to clipboard operation
vscode-yaml copied to clipboard

Published 20 hours ago • verified publisher icon redhat-developer

Fix added configuration setting for array indentation

Open msivasubramaniaan opened this issue 3 years ago • 1 comments

What does this PR do?

Allows users to configure the indentation size of the sequenceItem from it parent (i.e) sequence. The default is 0

image

What issues does this PR fix or reference?

https://github.com/redhat-developer/vscode-yaml/issues/172

Is it tested?

Yes

msivasubramaniaan avatar Nov 03 '21 05:11 msivasubramaniaan

While this can be useful, I do see a problem with it. We use prettier everywhere and its behavior is to indent sequences with two chars.

Now you can see why the proposed default might be problematic.

So I would be in favour of having a two char indentation by default or having a null default instead of 0 at least.

ssbarnea avatar May 18 '22 09:05 ssbarnea