razor icon indicating copy to clipboard operation
razor copied to clipboard

[FUSE] Folding range whitespace differences at the end of blocks

Open davidwengier opened this issue 1 year ago • 3 comments

Not sure if this is a tooling issue, or difference in results from Roslyn caused by a runtime/designtime compilation difference.

Example differences: image

From: https://github.com/dotnet/razor/commit/f929020b391baae015ded4a723f4d773c5d105df?diff=unified&w=0#diff-a1706b971c83e9c0c358e3db52e305ad63a07dc5a88c23d2ae265ce9e06eaf54L30

I think this is a bad user experience as, in the example above, the closing div tag would end up displayed on the same line as the @if when collapsed.

davidwengier avatar Sep 09 '24 22:09 davidwengier

https://github.com/dotnet/razor/pull/10959 marks an integration test as skipped referencing this issue. Please unskip when fixing :)

davidwengier avatar Oct 03 '24 23:10 davidwengier

@chsienki

jaredpar avatar Oct 08 '24 16:10 jaredpar

@jaredpar Yep, I mentioned this one previously. We've been fixing these on an ad-hoc basis but it turns out it's an issue with any @ directive that razor parses directly.

I plan to fix that holistically rather than doing more one at a time patches, but we're getting into the long tail of what users actually see so not considering this a blocker for fuse rollout right now.

(Aside, apologies I guess I didn't put these issues into milestones but have already triaged them).

chsienki avatar Oct 08 '24 17:10 chsienki

@chsienki @jaredpar Note: This issue is causing range folding problems for our customers and was reported in a VS feedback ticket

jordi1215 avatar Feb 28 '25 02:02 jordi1215