Mismatch between Prettier and indent-rainbow logic about vscode-indent-rainbow

[oderwat]

Indent Rainbow makes no assumptions about what is "pretty". Just disable error markers for the language or altogether, like so:

"indentRainbow.ignoreErrorLanguages" : [ "*" ]

[moeriki]

Nice ignoreErrorLanguages is perfect for my needs.

[gbrunow]

There is a slightly annoying thing that ignoring errors doesn't solve though, the highlight behind the first two characters.

[oderwat]

That doesn't annoy me as I would not have such kind of indent in my projects. But I guess I would accept a PR for an optional behavior if it doesn't degrade performance too much.

[gbrunow]

Personally I don't like that indentation either, but prettier is opinionated and won't let me change that, believe me, I tried.
I will take a look at the code and see if I can submit a PR request when I have time.

[gbrunow]

@oderwat I'm currently working on this and will probably submit other changes if that's okay with you.
I'm submitting a simple one in a few minutes just so it's not too much at once.

[oderwat]

After some more thinking about it, I decided that the current behavior is what I want. Resizing the bars to "fit" the code in the last example just looks wrong to me. Change the tab size to two if you write code in this style. You very obviously don't use an indentation of four spaces there.

[gbrunow]

It is 4 spaces but prettier isn't consistent when indenting chained methods.
This is what it looks with 2 spaces:

[keidarcy]

Does anyone know what's logic how rainbow-indent decides the space(2 or 4)?

[oderwat]

It uses what the editor uses. So it is either a "tab size" or count of "spaces". You can see that in the bottom status bar of the editor window.