fix(Wasm): Changing TextBox.Text in code should reset the caret

fix(Wasm): Changing TextBox.Text in code should reset the caret to the begging

fix(Wasm): Changing TextBox.Text in code should reset the caret to the begging

fix(Wasm): Changing TextBox.Text in code should reset the caret to the begging

fix(Wasm): Changing TextBox.Text in code should reset the caret to the beginning

fix(Wasm): Changing TextBox.Text in code should reset the caret to the beginning

fix(Wasm): Changing TextBox.Text in code should reset the caret to the beginning

fix(Wasm): Changing TextBox.Text in code should reset the caret to the begining

fix(Wasm): Changing TextBox.Text in code should reset the caret to the beginning

refactor: adjusted reset carret

refactor: setted just to WASM

refactor: resolved PR comments

refactor: commit PR recomendation.

refactor: Changed for specific code the WASM logic.

refactor: adjustes algorithm

fix(Wasm): Changing TextBox.Text in code should reset the caret to the beginning

refactor: adjusted reset carret

refactor: setted just to WASM

refactor: resolved PR comments

refactor: commit PR recomendation.

refactor: Changed for specific code the WASM logic.

refactor: resolved the comment on PR.

refactor: adjust resetcarret implementation

refactor: adjust unit test

refactor: adjust unit tests

refactor: adjust unit tests.

refactor: test adjustment

refactor: adjusted unit test.

refactor: adjuste android cursor position

Author: aesteves900

src/Uno.UI.Runtime.Skia.Wpf/Extensions/UI/Xaml/Controls/TextBoxViewExtension.cs

@@ -38,7 +38,11 @@ public void StartEntry()
 			_contentElement = textBox.ContentElement;
 
 			EnsureWidgetForAcceptsReturn();
-			textInputLayer.Children.Add(_currentInputWidget!);
+
+			if (textInputLayer.Children.Count == 0)
+			{
+				textInputLayer.Children.Add(_currentInputWidget!);
+			}
 
 			UpdateNativeView();
 			SetTextNative(textBox.Text);
@@ -160,7 +164,17 @@ public void SetIsPassword(bool isPassword)
 			// No support for now.
 		}
 
-		public void Select(int start, int length) => _currentInputWidget?.Select(start, length);
+		public void Select(int start, int length)
+		{
+			_currentInputWidget?.Select(start, length);
+
+			if (_currentInputWidget == null)
+			{
+				this.StartEntry();
+
+				_currentInputWidget!.Select(start, length);
+			}
+		}
 
 		public int GetSelectionStart() => _currentInputWidget?.SelectionStart ?? 0;
 

src/Uno.UI.RuntimeTests/Tests/Windows_UI_Xaml_Controls/Given_TextBox.cs

@@ -134,12 +134,8 @@ public async Task When_Calling_Select_With_In_Range_Values()
 
 			WindowHelper.WindowContent = textBox;
 			await WindowHelper.WaitForLoaded(textBox);
-#if __WASM__ // Wasm is behaving differently than UWP and other platforms. https://github.com/unoplatform/uno/issues/7016
-			Assert.AreEqual(10, textBox.SelectionStart);
-#else
-			Assert.AreEqual(0, textBox.SelectionStart);
-#endif
-
+			textBox.Focus(FocusState.Programmatic);
+			Assert.AreEqual(textBox.Text.Length, textBox.SelectionStart);
 			Assert.AreEqual(0, textBox.SelectionLength);
 			textBox.Select(1, 7);
 			Assert.AreEqual(1, textBox.SelectionStart);
@@ -156,12 +152,8 @@ public async Task When_Calling_Select_With_Out_Of_Range_Length()
 
 			WindowHelper.WindowContent = textBox;
 			await WindowHelper.WaitForLoaded(textBox);
-
-#if __WASM__ // Wasm is behaving differently than UWP and other platforms. https://github.com/unoplatform/uno/issues/7016
-			Assert.AreEqual(10, textBox.SelectionStart);
-#else
-			Assert.AreEqual(0, textBox.SelectionStart);
-#endif
+			textBox.Focus(FocusState.Programmatic);
+			Assert.AreEqual(textBox.Text.Length, textBox.SelectionStart);
 			Assert.AreEqual(0, textBox.SelectionLength);
 			textBox.Select(1, 20);
 			Assert.AreEqual(1, textBox.SelectionStart);
@@ -178,12 +170,8 @@ public async Task When_Calling_Select_With_Out_Of_Range_Start()
 
 			WindowHelper.WindowContent = textBox;
 			await WindowHelper.WaitForLoaded(textBox);
-
-#if __WASM__ // Wasm is behaving differently than UWP and other platforms. https://github.com/unoplatform/uno/issues/7016
-			Assert.AreEqual(10, textBox.SelectionStart);
-#else
-			Assert.AreEqual(0, textBox.SelectionStart);
-#endif
+			textBox.Focus(FocusState.Programmatic);
+			Assert.AreEqual(textBox.Text.Length, textBox.SelectionStart);
 			Assert.AreEqual(0, textBox.SelectionLength);
 			textBox.Select(20, 5);
 			Assert.AreEqual(10, textBox.SelectionStart);

src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.Android.cs

@@ -169,6 +169,15 @@ partial void OnFocusStateChangedPartial(FocusState focusState)
 					using (focusState == FocusState.Programmatic ? PreventKeyboardDisplayIfSet() : null)
 					{
 						_textBoxView.RequestFocus();
+
+						var selectionStart = this.SelectionStart;
+
+						if (selectionStart == 0)
+						{
+							int cursorPosition = selectionStart + _textBoxView?.Text?.Length ?? 0;
+
+							this.Select(cursorPosition, 0);
+						}
 					}
 				}
 			}

src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.cs

@@ -264,10 +264,12 @@ private void RaiseTextChanged()
 					_isTextChangedPending = false;
 				}
 			}
-
+			
 			_textBoxView?.SetTextNative(Text);
+		
 		}
 
+
 		private void UpdatePlaceholderVisibility()
 		{
 			if (_placeHolder != null)
@@ -303,9 +305,9 @@ private object CoerceText(object baseValue)
 			return baseString;
 		}
 
-		#endregion
+#endregion
 
-		#region Description DependencyProperty
+#region Description DependencyProperty
 
 		public
 #if __IOS__ || __MACOS__
@@ -342,7 +344,7 @@ private void UpdateDescriptionVisibility(bool initialization)
 				descriptionPresenter.Visibility = Description != null ? Visibility.Visible : Visibility.Collapsed;
 			}
 		}
-		#endregion
+#endregion
 
 		protected override void OnFontSizeChanged(double oldValue, double newValue)
 		{
@@ -374,7 +376,7 @@ protected override void OnFontWeightChanged(FontWeight oldValue, FontWeight newV
 
 		partial void OnForegroundColorChangedPartial(Brush newValue);
 
-		#region PlaceholderText DependencyProperty
+#region PlaceholderText DependencyProperty
 
 		public string PlaceholderText
 		{
@@ -390,9 +392,9 @@ public string PlaceholderText
 				new FrameworkPropertyMetadata(defaultValue: string.Empty)
 			);
 
-		#endregion
+#endregion
 
-		#region InputScope DependencyProperty
+#region InputScope DependencyProperty
 
 		public InputScope InputScope
 		{
@@ -423,9 +425,9 @@ public InputScope InputScope
 		protected void OnInputScopeChanged(DependencyPropertyChangedEventArgs e) => OnInputScopeChangedPartial(e);
 		partial void OnInputScopeChangedPartial(DependencyPropertyChangedEventArgs e);
 
-		#endregion
+#endregion
 
-		#region MaxLength DependencyProperty
+#region MaxLength DependencyProperty
 
 		public int MaxLength
 		{
@@ -448,9 +450,9 @@ public int MaxLength
 
 		partial void OnMaxLengthChangedPartial(DependencyPropertyChangedEventArgs e);
 
-		#endregion
+#endregion
 
-		#region AcceptsReturn DependencyProperty
+#region AcceptsReturn DependencyProperty
 
 		public bool AcceptsReturn
 		{
@@ -487,9 +489,9 @@ private void OnAcceptsReturnChanged(DependencyPropertyChangedEventArgs e)
 
 		partial void OnAcceptsReturnChangedPartial(DependencyPropertyChangedEventArgs e);
 
-		#endregion
+#endregion
 
-		#region TextWrapping DependencyProperty
+#region TextWrapping DependencyProperty
 		public TextWrapping TextWrapping
 		{
 			get => (TextWrapping)this.GetValue(TextWrappingProperty);
@@ -514,7 +516,7 @@ private void OnTextWrappingChanged(DependencyPropertyChangedEventArgs args)
 
 		partial void OnTextWrappingChangedPartial(DependencyPropertyChangedEventArgs e);
 
-		#endregion
+#endregion
 
 #if __IOS__ || NET461 || __WASM__ || __SKIA__ || __NETSTD_REFERENCE__ || __MACOS__
 		[Uno.NotImplemented("__IOS__", "NET461", "__WASM__", "__SKIA__", "__NETSTD_REFERENCE__", "__MACOS__")]
@@ -545,7 +547,7 @@ private void OnTextCharacterCasingChanged(DependencyPropertyChangedEventArgs e)
 
 		partial void OnTextCharacterCasingChangedPartial(DependencyPropertyChangedEventArgs e);
 
-		#region IsReadOnly DependencyProperty
+#region IsReadOnly DependencyProperty
 
 		public bool IsReadOnly
 		{
@@ -572,9 +574,9 @@ private void OnIsReadonlyChanged(DependencyPropertyChangedEventArgs e)
 
 		partial void OnIsReadonlyChangedPartial(DependencyPropertyChangedEventArgs e);
 
-		#endregion
+#endregion
 
-		#region Header DependencyProperties
+#region Header DependencyProperties
 
 		public object Header
 		{
@@ -618,9 +620,9 @@ private void OnHeaderChanged()
 			}
 		}
 
-		#endregion
+#endregion
 
-		#region IsSpellCheckEnabled DependencyProperty
+#region IsSpellCheckEnabled DependencyProperty
 
 		public bool IsSpellCheckEnabled
 		{
@@ -643,9 +645,9 @@ public bool IsSpellCheckEnabled
 
 		partial void OnIsSpellCheckEnabledChangedPartial(DependencyPropertyChangedEventArgs e);
 
-		#endregion
+#endregion
 
-		#region IsTextPredictionEnabled DependencyProperty
+#region IsTextPredictionEnabled DependencyProperty
 
 		[Uno.NotImplemented]
 		public bool IsTextPredictionEnabled
@@ -670,9 +672,9 @@ public bool IsTextPredictionEnabled
 
 		partial void OnIsTextPredictionEnabledChangedPartial(DependencyPropertyChangedEventArgs e);
 
-		#endregion
+#endregion
 
-		#region TextAlignment DependencyProperty
+#region TextAlignment DependencyProperty
 
 #if XAMARIN_ANDROID
 		public new TextAlignment TextAlignment
@@ -704,7 +706,7 @@ public string SelectedText
 					throw new ArgumentNullException();
 				}
 
-				var actual = (string)this.GetValue(TextProperty); 
+				var actual = (string)this.GetValue(TextProperty);
 				actual = actual.Remove(SelectionStart, SelectionLength);
 				actual = actual.Insert(SelectionStart, value);
 
@@ -908,7 +910,11 @@ private void DeleteButtonClick()
 
 		partial void OnDeleteButtonClickPartial();
 
-		internal void OnSelectionChanged() => SelectionChanged?.Invoke(this, new RoutedEventArgs(this));
+		internal void OnSelectionChanged()
+		{
+			SelectionChanged?.Invoke(this, new RoutedEventArgs(this));
+		}
+
 
 		public void OnTemplateRecycled()
 		{
@@ -989,7 +995,7 @@ internal override void UpdateThemeBindings(Data.ResourceUpdateReason updateReaso
 			UpdateKeyboardThemePartial();
 		}
 
-		partial void UpdateKeyboardThemePartial();
+		partial void UpdateKeyboardThemePartial();		
 
 		private protected override void OnIsEnabledChanged(IsEnabledChangedEventArgs e)
 		{
@@ -1000,10 +1006,10 @@ private protected override void OnIsEnabledChanged(IsEnabledChangedEventArgs e)
 
 		partial void OnIsEnabledChangedPartial(IsEnabledChangedEventArgs e);
 
-		private bool ShouldFocusOnPointerPressed(PointerRoutedEventArgs args)
+		private bool ShouldFocusOnPointerPressed(PointerRoutedEventArgs args) =>
 			// For mouse and pen, the TextBox should focus on pointer press
 			// (and then capture pointer to make sure to handle the whol down->move->up sequence).
 			// For touch we wait for the release to focus (avoid flickering in case of cancel due to scroll for instance).
-			=> args.Pointer.PointerDeviceType != PointerDeviceType.Touch;
+			args.Pointer.PointerDeviceType != PointerDeviceType.Touch;
 	}
 }

src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.wasm.cs

@@ -43,7 +43,7 @@ private void UpdateTextBoxView()
 				}
 			}
 		}
-
+		
 		partial void InitializePropertiesPartial()
 		{
 			if (_header != null)
@@ -59,6 +59,21 @@ partial void OnTappedPartial()
 			FocusTextView();
 		}
 
+
+		private void ResetCaret()
+		{
+
+			this.SelectionStart = 0;
+			this.SelectionLength = 0;
+
+			if (_textBoxView != null)
+			{
+				_textBoxView.SelectionStart = 0;
+				_textBoxView.SelectionEnd = 0;
+
+			}
+		}
+
 		private void OnHeaderClick(object sender, object args)
 		{
 			FocusTextView();

src/Uno.UI/UI/Xaml/Controls/TextBox/TextBoxView.skia.cs

@@ -87,11 +87,20 @@ internal void OnFocusStateChanged(FocusState focusState)
 			{
 				DisplayBlock.Opacity = 0;
 				_textBoxExtension?.StartEntry();
+				
+				var selectionStart = this.GetSelectionStart();
+
+				if (selectionStart == 0)
+				{
+					int cursorPosition = selectionStart + TextBox?.Text?.Length ?? 0;
+
+					_textBoxExtension?.Select(cursorPosition, 0);
+				}				
 			}
 			else
 			{
 				_textBoxExtension?.EndEntry();
-				DisplayBlock.Opacity = 1;
+				DisplayBlock.Opacity = 1;								
 			}
 		}