#25 Scroll to the cursor after document is formatted

Changelog.md

@@ -3,6 +3,7 @@
 * Fix: (#40) Typing performance issue
 * Fix: Some commands (e.g. comment / uncomment / format table) is executed with delay
 * Fix: Steps are reported as undefined for newly opened project and for non-project feature files
+* Fix: (#25) Scroll position in code editor is reset when reformatting feature file
 
 # v2021.4.5 - 2021-12-30
 

SpecFlow.VisualStudio/Editor/Commands/AutoFormatDocumentCommand.cs

@@ -77,7 +77,9 @@ public override bool PreExec(IWpfTextView textView, DeveroomEditorCommandTargetK
             textEdit.Apply();
         }
 
-        textView.Caret.MoveTo(textView.TextSnapshot.GetLineFromLineNumber(caretLineNumber).End);
+        var lineFromLineNumber = textView.TextSnapshot.GetLineFromLineNumber(caretLineNumber);
+        textView.Caret.MoveTo(lineFromLineNumber.End);
+        textView.ViewScroller.EnsureSpanVisible(lineFromLineNumber.Extent);
 
         return true;
     }

Tests/SpecFlow.VisualStudio.Specs/StepDefinitions/ProjectSystemSteps.cs

@@ -37,7 +37,7 @@ public void GivenThereIsANon_SpecFlowProjectScope()
     {
         CreateProject(ps => ps.StubProjectSettingsProvider.Kind = DeveroomProjectKind.FeatureFileContainerProject);
     }
-    
+
     [Given("there is a project scope which is (.*)")]
     public void GivenThereIsAProjectScopeWhichIs(DeveroomProjectKind kind)
     {

Tests/SpecFlow.VisualStudio.VsxStubs/StubViewScroller.cs

@@ -0,0 +1,34 @@
+namespace SpecFlow.VisualStudio.VsxStubs;
+
+public class StubViewScroller : IViewScroller
+{
+    public void ScrollViewportVerticallyByPixels(double distanceToScroll)
+    {
+    }
+
+    public void ScrollViewportVerticallyByLine(ScrollDirection direction)
+    {
+    }
+
+    public void ScrollViewportVerticallyByLines(ScrollDirection direction, int count)
+    {
+    }
+
+    public bool ScrollViewportVerticallyByPage(ScrollDirection direction) => false;
+
+    public void ScrollViewportHorizontallyByPixels(double distanceToScroll)
+    {
+    }
+
+    public void EnsureSpanVisible(SnapshotSpan span)
+    {
+    }
+
+    public void EnsureSpanVisible(SnapshotSpan span, EnsureSpanVisibleOptions options)
+    {
+    }
+
+    public void EnsureSpanVisible(VirtualSnapshotSpan span, EnsureSpanVisibleOptions options)
+    {
+    }
+}

Tests/SpecFlow.VisualStudio.VsxStubs/StubWpfTextView.cs

@@ -11,6 +11,7 @@ public StubWpfTextView(ITextBuffer textBuffer)
         TextBuffer = textBuffer;
         _caret = new StubTextCaret(this);
         Selection = new StubTextSelection(this);
+        ViewScroller = new StubViewScroller();
     }
 
     public StubEditorOptions StubEditorOptions { get; } = new();
@@ -73,7 +74,7 @@ ITextViewLine ITextView.GetTextViewLineContainingBufferPosition(SnapshotPoint bu
 
     public bool InLayout => true; //in a process of layout
 
-    public IViewScroller ViewScroller => throw new NotImplementedException();
+    public IViewScroller ViewScroller { get; }
 
     public IWpfTextViewLineCollection TextViewLines => throw new NotImplementedException();