Razor editor format document indents incorrectly for InputSelect components

[vsfeedback]

This issue has been moved from a ticket on Developer Community.

---

When using Format Document (Ctrl E+D), the resulting format of razor files sometimes produces incorrect indentation. This seems to happen where code switches between markup and code.

In the code in the screenshot for example, there are a couple of inconsistencies.

1. The braces after the foreach loops get indented differently.
2. The closing tags of the InputSelects end up with indentations not always matching the opening tag.

The second might issue be related to the first, if for example the formatter wants to have the next line after the closing brace to be two tabs behind the brace. Either way, this is not correct.

In my perfect world, the braces would be indented like in the first foreach loop in the screenshot, with the braces at the same level as the foreach keyword, but with the contents of the loop being indented one step. And of course the closing InputSelect tag should be at the same level as the opening tag.

---

Original Comments

Feedback Bot on 26/10/2021, 01:02 PM:

We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

---

Original Solutions

(no solutions)

[davidwengier]

This seems to repro with an InputSelect component. There is one in the eShopOnWeb project if anyone is looking :)

[davidwengier]

Additional reports:
https://developercommunity.visualstudio.com/t/Blazor-format-document---Formatting-goes/1560559
https://developercommunity.visualstudio.com/t/Blazor-Format-Document-doesnt-work-as-e/1575423

[davidwengier]

This repros it:

<div>
    @if (true)
    {
        <div>
            <InputSelect @bind-Value="_item.CatalogBrandId">
                @if (true)
                {
                    <option>goo</option>
                }
            </InputSelect>
        </div>
    }
</div>

Without the outer if, or without the @bind-value, it doesn't repro

[davidwengier]

Looks like this is caused by a bug in the Roslyn C# formatter: dotnet/roslyn#57465

[davidwengier]

Another report:
https://developercommunity.visualstudio.com/t/Poor-HTML-alignment-with-Blazor-componen/1660959

[davidwengier]

Another report:
https://developercommunity.visualstudio.com/t/CtrlKD-does-not-work-for-blazor-InputS/1665709

[davidwengier]

The Roslyn issue is fixed! Will leave this open until I have been able to verify the fix.

[davidwengier]

Update, this is now blocked by dotnet/roslyn#60437