Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Razor LSP: Fix and add tests for known formatting issues #5123

Closed
10 of 13 tasks
ajaybhargavb opened this issue Oct 2, 2020 · 1 comment
Closed
10 of 13 tasks

Razor LSP: Fix and add tests for known formatting issues #5123

ajaybhargavb opened this issue Oct 2, 2020 · 1 comment
Assignees
Milestone

Comments

@ajaybhargavb
Copy link
Contributor

Below is a list of known issues so far,

  • Cleanup doesn't work well in some cases
@if (true) { <div></div> }
    @{
        @:is
    }
<label for="email" @if (true)
                            {<text>a="b"</text>}>c</label>
  • Html attribute formatting not maintained
<textarea name="foo"
          id="foo">@("Foo")
        test</textarea>
@if (true)
{
    @if (true)
    {
        ...
    }
}
@code {
    public class Foo { }
}
<div>
    <div>
    @if(true)
    {
    }
    
    </div>
</div>
@{
    var x = DateTime
            .Now;
}
@ajaybhargavb ajaybhargavb self-assigned this Oct 2, 2020
@ajaybhargavb
Copy link
Contributor Author

Only 3 more issues remain from the above list and are tracked separately. Closing this.

@ghost ghost locked as resolved and limited conversation to collaborators Feb 11, 2021
@ghost ghost unlocked this conversation Oct 8, 2021
@allisonchou allisonchou transferred this issue from dotnet/aspnetcore Oct 8, 2021
@allisonchou allisonchou added this to the 16.9-Preview4 milestone Oct 8, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Nov 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants