Fix Android StaticLayout crash with comprehensive negative width prevention

Co-authored-by: PaulAndersonS <[email protected]>

Author: Copilot

maui/src/Core/Extensions/CanvasExtensions.Android.cs

@@ -2,6 +2,7 @@
 using Android.Graphics;
 using Android.Text;
 using Microsoft.Maui.Graphics.Platform;
+using System;
 using Font = Microsoft.Maui.Font;
 using Paint = Android.Graphics.Paint;
 using Rect = Microsoft.Maui.Graphics.Rect;
@@ -106,7 +107,9 @@ public static void DrawText(this ICanvas canvas, string value, Rect rect, Horizo
 						alignment = Android.Text.Layout.Alignment.AlignOpposite;
 					}
 
-					StaticTextLayout layout = new StaticTextLayout(value, paint, (int)(rect.Width * nativeCanvas.DisplayScale), alignment, 1.0f, 0.0f, false);
+					// Ensure width is positive to prevent Android StaticLayout crashes
+					int layoutWidth = Math.Max(1, (int)(rect.Width * nativeCanvas.DisplayScale));
+					StaticTextLayout layout = new StaticTextLayout(value, paint, layoutWidth, alignment, 1.0f, 0.0f, false);
 					double rectDensityHeight = rect.Height * nativeCanvas.DisplayScale;
 					//// Check the layout does not accommodate the text inside the specified rect then
 					//// restrict the text rendering with line count.

maui/src/TextInputLayout/SfTextInputLayout.Methods.cs

@@ -4,6 +4,7 @@
 using Syncfusion.Maui.Toolkit.NumericUpDown;
 using Syncfusion.Maui.Toolkit.NumericEntry;
 using Syncfusion.Maui.Toolkit.Themes;
+using System;
 using Path = Microsoft.Maui.Controls.Shapes.Path;
 
 namespace Syncfusion.Maui.Toolkit.TextInputLayout
@@ -1044,7 +1045,7 @@ void UpdateContentPosition()
 				_viewBounds.X = (int)_leadViewWidth;
 				UpdatePosition();
 				_viewBounds.Y = 0;
-				_viewBounds.Width = (int)(Width - _leadViewWidth - _trailViewWidth);
+				_viewBounds.Width = (int)Math.Max(1, Width - _leadViewWidth - _trailViewWidth);
 				_viewBounds.Height = (int)Height;
 
 				if (EnablePasswordVisibilityToggle && !ShowUpDownButton)
@@ -1061,6 +1062,9 @@ void UpdateContentPosition()
 					_viewBounds.Width -= (float)(IconSize * (IsUpDownVerticalAlignment ? 1 : 2));
 				}
 
+				// Ensure width doesn't become negative after all subtractions
+				_viewBounds.Width = (float)Math.Max(1, _viewBounds.Width);
+
 				if (_viewBounds.Height >= 0 && _viewBounds.Width >= 0)
 				{
 					AbsoluteLayout.SetLayoutBounds(Content, _viewBounds);
@@ -1454,7 +1458,7 @@ void UpdateHelperTextPosition()
 				_helperTextRect.Y = (int)(Height - TotalAssistiveTextHeight());
 			}
 
-			_helperTextRect.Width = (int)(Width - startPadding - CounterTextPadding - DefaultAssistiveLabelPadding - ((ShowCharCount) ? CounterTextSize.Width + CounterTextPadding : 0) - _trailViewWidth - _leadViewWidth);
+			_helperTextRect.Width = (int)Math.Max(1, Width - startPadding - CounterTextPadding - DefaultAssistiveLabelPadding - ((ShowCharCount) ? CounterTextSize.Width + CounterTextPadding : 0) - _trailViewWidth - _leadViewWidth);
 			_helperTextRect.Height = HelperTextSize.Height;
 
 			if (IsRTL)
@@ -1471,11 +1475,13 @@ double GetAssistiveTextWidth()
 
 			if (Width >= 0)
 			{
-				return Width - (IsNone ? 0 : StartX + DefaultAssistiveLabelPadding) - CounterTextPadding - DefaultAssistiveLabelPadding - ((ShowCharCount) ? CounterTextSize.Width + CounterTextPadding : 0) - _trailViewWidth - _leadViewWidth;
+				var calculatedWidth = Width - (IsNone ? 0 : StartX + DefaultAssistiveLabelPadding) - CounterTextPadding - DefaultAssistiveLabelPadding - ((ShowCharCount) ? CounterTextSize.Width + CounterTextPadding : 0) - _trailViewWidth - _leadViewWidth;
+				return Math.Max(1, calculatedWidth);
 			}
 			else if (WidthRequest != -1)
 			{
-				return WidthRequest - (IsNone ? 0 : StartX + DefaultAssistiveLabelPadding) - CounterTextPadding - DefaultAssistiveLabelPadding - ((ShowCharCount) ? CounterTextSize.Width + CounterTextPadding : 0) - _trailViewWidth - _leadViewWidth;
+				var calculatedWidth = WidthRequest - (IsNone ? 0 : StartX + DefaultAssistiveLabelPadding) - CounterTextPadding - DefaultAssistiveLabelPadding - ((ShowCharCount) ? CounterTextSize.Width + CounterTextPadding : 0) - _trailViewWidth - _leadViewWidth;
+				return Math.Max(1, calculatedWidth);
 			}
 			else
 			{
@@ -1495,11 +1501,13 @@ double GetHintTextWidth()
 
 			if (Width >= 0)
 			{
-				return Width - (IsNone ? 0 : ((2 * StartX) + DefaultAssistiveLabelPadding)) - _trailViewWidth - _leadViewWidth;
+				var calculatedWidth = Width - (IsNone ? 0 : ((2 * StartX) + DefaultAssistiveLabelPadding)) - _trailViewWidth - _leadViewWidth;
+				return Math.Max(1, calculatedWidth);
 			}
 			else if (WidthRequest != -1)
 			{
-				return WidthRequest - (IsNone ? 0 : 2 * (StartX + DefaultAssistiveLabelPadding)) - _trailViewWidth - _leadViewWidth;
+				var calculatedWidth = WidthRequest - (IsNone ? 0 : 2 * (StartX + DefaultAssistiveLabelPadding)) - _trailViewWidth - _leadViewWidth;
+				return Math.Max(1, calculatedWidth);
 			}
 			else
 			{
@@ -1525,7 +1533,7 @@ void UpdateErrorTextPosition()
 				_errorTextRect.Y = (int)(Height - TotalAssistiveTextHeight());
 			}
 
-			_errorTextRect.Width = (int)(Width - startPadding - CounterTextPadding - DefaultAssistiveLabelPadding - ((ShowCharCount) ? CounterTextSize.Width + CounterTextPadding : 0) - _trailViewWidth - _leadViewWidth);
+			_errorTextRect.Width = (int)Math.Max(1, Width - startPadding - CounterTextPadding - DefaultAssistiveLabelPadding - ((ShowCharCount) ? CounterTextSize.Width + CounterTextPadding : 0) - _trailViewWidth - _leadViewWidth);
 			_errorTextRect.Height = ErrorTextSize.Height;
 
 			if (IsRTL)
@@ -1585,11 +1593,11 @@ void UpdateOutlineRectF()
 			UpdateTrailViewWidthForBorder();
 			if (BaseLineMaxHeight <= 2)
 			{
-				_outlineRectF.Width = (float)((Width - (BaseLineMaxHeight * 2)) - _leadViewWidth - _trailViewWidth);
+				_outlineRectF.Width = (float)Math.Max(1, (Width - (BaseLineMaxHeight * 2)) - _leadViewWidth - _trailViewWidth);
 			}
 			else
 			{
-				_outlineRectF.Width = (float)((Width - (BaseLineMaxHeight)) - _leadViewWidth - _trailViewWidth);
+				_outlineRectF.Width = (float)Math.Max(1, (Width - (BaseLineMaxHeight)) - _leadViewWidth - _trailViewWidth);
 			}
 
 			_outlineRectF.Height = (float)(Height - _outlineRectF.Y - TotalAssistiveTextHeight() - AssistiveLabelPadding);
@@ -1623,7 +1631,7 @@ void UpdateBackgroundRectF()
 			UpdateTrailViewWidthForBorder();
 			_backgroundRectF.X = (float)_leadViewWidth;
 			_backgroundRectF.Y = 0;
-			_backgroundRectF.Width = (float)(Width - _leadViewWidth - _trailViewWidth);
+			_backgroundRectF.Width = (float)Math.Max(1, Width - _leadViewWidth - _trailViewWidth);
 			if (BaseLineMaxHeight <= 2)
 			{
 				_backgroundRectF.Height = (float)(Height - TotalAssistiveTextHeight() - AssistiveLabelPadding);

maui/tests/Syncfusion.Maui.Toolkit.UnitTest/Layout/SfTextInputLayoutUnitTests.cs

@@ -1236,6 +1236,27 @@ public void VerySmallContainerWidthWithLeadingAndTrailingViews_ShouldNotCrash()
 			Assert.True(inputLayout.ShowTrailingView);
 		}
 
+		[Fact]
+		public void VerySmallContainerWidthWithErrorAndHelperText_ShouldNotCrash()
+		{
+			// Test for the fix where very small container widths with error/helper text cause crashes
+			var inputLayout = new SfTextInputLayout
+			{
+				Content = new Entry { Text = "Test" },
+				Hint = "Name",
+				ErrorText = "This is an error message",
+				HelperText = "This is a helper message",
+				WidthRequest = 10, // Very small width that could cause negative text layout bounds
+				HeightRequest = 50
+			};
+
+			// The control should handle very small dimensions gracefully without throwing exceptions
+			Assert.NotNull(inputLayout);
+			Assert.NotNull(inputLayout.Content);
+			Assert.Equal("This is an error message", inputLayout.ErrorText);
+			Assert.Equal("This is a helper message", inputLayout.HelperText);
+		}
+
 
 		#endregion
 	}