Merge pull request #11544 from robloo/RangeBaseValueChangedEventArgs

Switch RangeBase.ValueChanged Event Args Type

src/Avalonia.Base/Interactivity/RoutedPropertyChangedEventArgs.cs

@@ -1,46 +0,0 @@
-namespace Avalonia.Interactivity
-{
-    /// <summary>
-    /// Provides both old and new property values with a routed event.
-    /// </summary>
-    /// <typeparam name="T">The type of values.</typeparam>
-    public class RoutedPropertyChangedEventArgs<T> : RoutedEventArgs
-    {
-        /// <summary>
-        /// Initializes a new instance of the <see cref="RoutedPropertyChangedEventArgs{T}"/> class.
-        /// </summary>
-        /// <param name="oldValue">The old property value.</param>
-        /// <param name="newValue">The new property value.</param>
-        /// <param name="routedEvent">The routed event associated with these event args.</param>
-        public RoutedPropertyChangedEventArgs(T oldValue, T newValue, RoutedEvent? routedEvent)
-            : base(routedEvent)
-        {
-            OldValue = oldValue;
-            NewValue = newValue;
-        }
-
-        /// <summary>
-        /// Initializes a new instance of the <see cref="RoutedPropertyChangedEventArgs{T}"/> class.
-        /// </summary>
-        /// <param name="oldValue">The old property value.</param>
-        /// <param name="newValue">The new property value.</param>
-        /// <param name="routedEvent">The routed event associated with these event args.</param>
-        /// <param name="source">The source object that raised the routed event.</param>
-        public RoutedPropertyChangedEventArgs(T oldValue, T newValue, RoutedEvent? routedEvent, object? source)
-            : base(routedEvent, source)
-        {
-            OldValue = oldValue;
-            NewValue = newValue;
-        }
-
-        /// <summary>
-        /// Gets the old value of the property.
-        /// </summary>
-        public T OldValue { get; init; }
-
-        /// <summary>
-        /// Gets the new value of the property.
-        /// </summary>
-        public T NewValue { get; init; }
-    }
-}

src/Avalonia.Controls/Primitives/RangeBase.cs

@@ -45,14 +45,14 @@ namespace Avalonia.Controls.Primitives
         /// <summary>
         /// Defines the <see cref="ValueChanged"/> event.
         /// </summary>
-        public static readonly RoutedEvent<RoutedPropertyChangedEventArgs<double>> ValueChangedEvent =
-            RoutedEvent.Register<RangeBase, RoutedPropertyChangedEventArgs<double>>(
+        public static readonly RoutedEvent<RangeBaseValueChangedEventArgs> ValueChangedEvent =
+            RoutedEvent.Register<RangeBase, RangeBaseValueChangedEventArgs>(
                 nameof(ValueChanged), RoutingStrategies.Bubble);
 
         /// <summary>
         /// Occurs when the <see cref="Value"/> property changes.
         /// </summary>
-        public event EventHandler<RoutedPropertyChangedEventArgs<double>>? ValueChanged
+        public event EventHandler<RangeBaseValueChangedEventArgs>? ValueChanged
         {
             add => AddHandler(ValueChangedEvent, value);
             remove => RemoveHandler(ValueChangedEvent, value);
@@ -163,7 +163,7 @@ namespace Avalonia.Controls.Primitives
             }
             else if (change.Property == ValueProperty)
             {
-                var valueChangedEventArgs = new RoutedPropertyChangedEventArgs<double>(
+                var valueChangedEventArgs = new RangeBaseValueChangedEventArgs(
                     change.GetOldValue<double>(),
                     change.GetNewValue<double>(),
                     ValueChangedEvent);

src/Avalonia.Controls/Primitives/RangeBaseValueChangedEventArgs.cs

@@ -0,0 +1,47 @@
+using Avalonia.Interactivity;
+
+namespace Avalonia.Controls.Primitives
+{
+    /// <summary>
+    /// Provides data specific to a <see cref="RangeBase.ValueChanged"/> event.
+    /// </summary>
+    public class RangeBaseValueChangedEventArgs : RoutedEventArgs
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="RangeBaseValueChangedEventArgs"/> class.
+        /// </summary>
+        /// <param name="oldValue">The old value of the range value property.</param>
+        /// <param name="newValue">The new value of the range value property.</param>
+        /// <param name="routedEvent">The routed event associated with these event args.</param>
+        public RangeBaseValueChangedEventArgs(double oldValue, double newValue, RoutedEvent? routedEvent)
+            : base(routedEvent)
+        {
+            OldValue = oldValue;
+            NewValue = newValue;
+        }
+
+        /// <summary>
+        /// Initializes a new instance of the <see cref="RangeBaseValueChangedEventArgs"/> class.
+        /// </summary>
+        /// <param name="oldValue">The old value of the range value property.</param>
+        /// <param name="newValue">The new value of the range value property.</param>
+        /// <param name="routedEvent">The routed event associated with these event args.</param>
+        /// <param name="source">The source object that raised the routed event.</param>
+        public RangeBaseValueChangedEventArgs(double oldValue, double newValue, RoutedEvent? routedEvent, object? source)
+            : base(routedEvent, source)
+        {
+            OldValue = oldValue;
+            NewValue = newValue;
+        }
+
+        /// <summary>
+        /// Gets the old value of the range value property.
+        /// </summary>
+        public double OldValue { get; init; }
+
+        /// <summary>
+        /// Gets the new value of the range value property.
+        /// </summary>
+        public double NewValue { get; init; }
+    }
+}

src/Avalonia.Controls/TextChangedEventArgs.cs

@@ -3,15 +3,24 @@
 namespace Avalonia.Controls
 {
     /// <summary>
-    /// Provides data specific to a TextChanged event.
+    /// Provides data specific to a <see cref="TextBox.TextChanged"/> event.
     /// </summary>
     public class TextChangedEventArgs : RoutedEventArgs
     {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="TextChangedEventArgs"/> class.
+        /// </summary>
+        /// <param name="routedEvent">The routed event associated with these event args.</param>
         public TextChangedEventArgs(RoutedEvent? routedEvent)
             : base (routedEvent)
         {
         }
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="TextChangedEventArgs"/> class.
+        /// </summary>
+        /// <param name="routedEvent">The routed event associated with these event args.</param>
+        /// <param name="source">The source object that raised the routed event.</param>
         public TextChangedEventArgs(RoutedEvent? routedEvent, Interactive? source)
             : base(routedEvent, source)
         {

src/Avalonia.Controls/TextChangingEventArgs.cs

@@ -3,15 +3,24 @@
 namespace Avalonia.Controls
 {
     /// <summary>
-    /// Provides data specific to a TextChanging event.
+    /// Provides data specific to a <see cref="TextBox.TextChanging"/> event.
     /// </summary>
     public class TextChangingEventArgs : RoutedEventArgs
     {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="TextChangingEventArgs"/> class.
+        /// </summary>
+        /// <param name="routedEvent">The routed event associated with these event args.</param>
         public TextChangingEventArgs(RoutedEvent? routedEvent)
             : base (routedEvent)
         {
         }
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="TextChangingEventArgs"/> class.
+        /// </summary>
+        /// <param name="routedEvent">The routed event associated with these event args.</param>
+        /// <param name="source">The source object that raised the routed event.</param>
         public TextChangingEventArgs(RoutedEvent? routedEvent, Interactive? source)
             : base(routedEvent, source)
         {