Merge pull request #175 from syncfusion/BottomSheetAnimationDuration

Provide animation duration support for Bottom sheet control

Author: PaulAndersonS

maui/src/BottomSheet/SfBottomSheet.cs

@@ -489,6 +489,19 @@ public partial class SfBottomSheet : SfView, IParentThemeElement
 			BindingMode.Default
 			);
 
+		/// <summary>
+		/// Identifies the <see cref="AnimationDuration"/> bindable property.
+		/// </summary>
+		/// <value>
+		/// The identifier for <see cref="AnimationDuration"/> bindable property.
+		/// </value>
+		public static readonly BindableProperty AnimationDurationProperty = BindableProperty.Create(
+			nameof(AnimationDuration),
+			typeof(double),
+			typeof(SfBottomSheet),
+			150d,
+			BindingMode.Default);
+
 		#endregion
 
 		#region Internal Bindable Properties
@@ -1183,6 +1196,19 @@ public bool CollapseOnOverlayTap
 			set => SetValue(CollapseOnOverlayTapProperty, value);
 		}
 
+		/// <summary> 
+		/// Gets or sets the duration, in milliseconds, for the opening and closing animations.
+		/// </summary> 
+		/// <value> 
+		/// It accepts double values, and the default value is 150ms.
+		/// </value> 
+
+		public double AnimationDuration
+		{
+			get => (double)GetValue(AnimationDurationProperty);
+			set => SetValue(AnimationDurationProperty, value);
+		}
+
 		#endregion
 
 		#region Internal Properties
@@ -1256,6 +1282,17 @@ public void Close()
 		    }
 		}
 
+		/// <summary>
+		/// Returns the value of <c>AnimationDuration</c>, ensuring it is clamped to a non-negative integer.
+		/// This method is useful when passing the duration to animation APIs that require a <c>uint</c> value,
+		/// preventing issues caused by negative durations.
+		/// </summary>
+		/// <returns>A non-negative integer representing the animation duration.</returns>
+		int GetClampedAnimationDuration()
+		{
+			return (int)Math.Max(0, AnimationDuration);
+		}
+
 		#endregion
 
 		#region Internal Methods
@@ -2093,13 +2130,13 @@ void AnimateBottomSheet(double targetPosition, Action? onFinish = null)
 				_overlayGrid.AbortAnimation("overlayGridAnimation");
 			}
 
-			const int animationDuration = 150;
-		    const int topPadding = 2;
+			int animationDuration = this.GetClampedAnimationDuration();
+			const int topPadding = 2;
 			_isSheetOpen = true;
 			if (_bottomSheet is not null)
 			{
 				var bottomSheetAnimation = new Animation(d => _bottomSheet.TranslationY = d, _bottomSheet.TranslationY, targetPosition + topPadding);
-				_bottomSheet?.Animate("bottomSheetAnimation", bottomSheetAnimation, length: animationDuration, easing: Easing.Linear, finished: (v, e) =>
+				_bottomSheet?.Animate("bottomSheetAnimation", bottomSheetAnimation, length: (uint)animationDuration, easing: Easing.Linear, finished: (v, e) => 
 				{
 					UpdateBottomSheetHeight();
 					onFinish?.Invoke();
@@ -2133,7 +2170,15 @@ void AnimateOverlay(int animationDuration)
 						endValue = DefaultOverlayOpacity;
 					}
 
-					var overlayGridAnimation = new Animation(d => _overlayGrid.Opacity = d, startValue, endValue);
+					var overlayGridAnimation = new Animation(d =>
+					{
+						// Ensure the opacity is only updated with valid numeric values to avoid rendering issues.
+						if (!double.IsNaN(d))
+						{
+							_overlayGrid.Opacity = d;
+						}
+					}
+					, startValue, endValue);
 					_overlayGrid.Animate("overlayGridAnimation", overlayGridAnimation,
 						length: (uint)animationDuration,
 						easing: Easing.Linear,

maui/tests/Syncfusion.Maui.Toolkit.UnitTest/Navigation/SfBottomSheetUnitTests.cs

@@ -38,6 +38,7 @@ public void Constructor_InitializesDefaultsCorrectly()
 			Assert.Equal(4d, _bottomSheet.GrabberHeight);
 			Assert.Equal(32d, _bottomSheet.GrabberWidth);
 			Assert.Equal(12d, _bottomSheet.GrabberCornerRadius);
+			Assert.Equal(150d, _bottomSheet.AnimationDuration);
 			if (_bottomSheet.GrabberBackground is SolidColorBrush grabberBrush)
 			{
 				var grabberColor = grabberBrush.Color;
@@ -475,6 +476,19 @@ public void CollapseOnOverlayTap(bool input, bool expected)
 			Assert.Equal(expected, actual);
 		}
 
+		[Theory]
+		[InlineData(500d, 500d)]
+		[InlineData(0d, 0d)]
+		[InlineData(-500d, -500d)]
+		public void AnimationDurationProperty_ReturnsSetValue(double input, double expected)
+		{
+			_bottomSheet.AnimationDuration = input;
+
+			var actual = _bottomSheet.AnimationDuration;
+
+			Assert.Equal(expected, actual);
+		}
+
 		#endregion
 
 		#region Internal Properties
@@ -733,6 +747,14 @@ public void UpdateGrabberHeightProperty(double input, double expected)
 			Assert.Equal(expected, actual);
 		}
 
+		[Fact]
+		public void GetClampedAnimationDuration()
+		{
+			_bottomSheet.AnimationDuration = -500;
+			var actual = InvokePrivateMethod(_bottomSheet, "GetClampedAnimationDuration");
+			Assert.Equal(0, actual);
+		}
+
 		[Theory]
 		[InlineData(48, 48)]
 		[InlineData(0, 32)]