Merge pull request #362 from FrozenAssassine/fixfor#356

FrozenAssassine · web-flow · commit b2605673ec24 · 2023-01-24T16:51:56.000+01:00
This fixes #356
diff --git a/TextControlBox/Renderer/SelectionRenderer.cs b/TextControlBox/Renderer/SelectionRenderer.cs
@@ -101,12 +101,18 @@ public TextSelection DrawSelection(CanvasTextLayout TextLayout, IEnumerable<stri
                 }
 
                 SelectionStart = Math.Min(SelStartIndex, SelEndIndex);
+                
+                if (SelectionStart < 0)
+                    SelectionStart = 0;
+                if (SelectionLength < 0)
+                    SelectionLength = 0;
 
                 if (SelEndIndex > SelStartIndex)
                     SelectionLength = SelEndIndex - SelStartIndex;
                 else
                     SelectionLength = SelStartIndex - SelEndIndex;
 
+                Debug.WriteLine("SELSTART: " + SelectionStart);
                 CanvasTextLayoutRegion[] descriptions = TextLayout.GetCharacterRegions(SelectionStart, SelectionLength);
                 for (int i = 0; i < descriptions.Length; i++)
                 {
diff --git a/TextControlBox/Text/CursorPosition.cs b/TextControlBox/Text/CursorPosition.cs
@@ -29,6 +29,8 @@ internal void AddToCharacterPos(int Add)
         internal void SubtractFromCharacterPos(int Value)
         {
             CharacterPosition -= Value;
+            if (CharacterPosition < 0)
+                CharacterPosition = 0;
         }
         internal new string ToString()
         {

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,8 @@ internal void AddToCharacterPos(int Add)`
`29`	`29`	`internal void SubtractFromCharacterPos(int Value)`
`30`	`30`	`{`
`31`	`31`	`CharacterPosition -= Value;`
	`32`	`+ if (CharacterPosition < 0)`
	`33`	`+ CharacterPosition = 0;`
`32`	`34`	`}`
`33`	`35`	`internal new string ToString()`
`34`	`36`	`{`