Ensure found text is always visible

Fixes #44
fernandreu · Jul 14, 2019 · a3b9b00 · a3b9b00
1 parent 095a5fe
commit a3b9b00
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/OfficeRibbonXEditor/ViewModels/FindReplaceDialogViewModel.cs b/OfficeRibbonXEditor/ViewModels/FindReplaceDialogViewModel.cs
@@ -359,6 +359,10 @@ private void FindWrapper(bool searchUp)
                     this.StatusText = $"Search match wrapped to the beginning of the {(this.SearchSelection ? "selection" : "document")}";
                 }
 
+                // This should ensure the entire text is visible before it is selected
+                this.Scintilla.GotoPosition(foundRange.cpMax);
+                this.Scintilla.GotoPosition(foundRange.cpMin);
+
                 this.Scintilla.SetSel(foundRange.cpMin, foundRange.cpMax);
                 this.MoveDialogAwayFromSelection();
             }
@@ -464,8 +468,13 @@ private void ReplaceWrapper(bool searchUp)
                 {
                     this.StatusText = $"Search match wrapped to the beginning of the {(this.SearchSelection ? "selection" : "document")}";
                 }
+
+                // This should ensure the entire text is visible before it is selected
+                this.Scintilla.GotoPosition(nextRange.cpMax);
+                this.Scintilla.GotoPosition(nextRange.cpMin);
 
                 this.Scintilla.SetSel(nextRange.cpMin, nextRange.cpMax);
+
                 this.MoveDialogAwayFromSelection();
             }