Rename TValue type params to TElement

Author: Sergio0694

CommunityToolkit.Mvvm/Collections/ObservableGroupedCollection{TKey,TValue}.cs renamed to CommunityToolkit.Mvvm/Collections/ObservableGroupedCollection{TKey,TElement}.cs

@@ -13,9 +13,9 @@ namespace CommunityToolkit.Mvvm.Collections;
 /// <summary>
 /// An observable list of observable groups.
 /// </summary>
-/// <typeparam name="TKey">The type of the group key.</typeparam>
-/// <typeparam name="TValue">The type of the items in the collection.</typeparam>
-public sealed class ObservableGroupedCollection<TKey, TValue> : ObservableCollection<ObservableGroup<TKey, TValue>>
+/// <typeparam name="TKey">The type of the group keys.</typeparam>
+/// <typeparam name="TElement">The type of elements in the collection.</typeparam>
+public sealed class ObservableGroupedCollection<TKey, TElement> : ObservableCollection<ObservableGroup<TKey, TElement>>
     where TKey : notnull
 {
     /// <summary>
@@ -29,8 +29,8 @@ public ObservableGroupedCollection()
     /// Initializes a new instance of the <see cref="ObservableGroupedCollection{TKey, TValue}"/> class.
     /// </summary>
     /// <param name="collection">The initial data to add in the grouped collection.</param>
-    public ObservableGroupedCollection(IEnumerable<IGrouping<TKey, TValue>> collection)
-        : base(collection.Select(static c => new ObservableGroup<TKey, TValue>(c)))
+    public ObservableGroupedCollection(IEnumerable<IGrouping<TKey, TElement>> collection)
+        : base(collection.Select(static c => new ObservableGroup<TKey, TElement>(c)))
     {
     }
 
@@ -40,9 +40,9 @@ public ObservableGroupedCollection(IEnumerable<IGrouping<TKey, TValue>> collecti
     /// <param name="list">The resulting <see cref="List{T}"/>, if one was in use.</param>
     /// <returns>Whether or not a <see cref="List{T}"/> instance has been found.</returns>
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    internal bool TryGetList([NotNullWhen(true)] out List<ObservableGroup<TKey, TValue>>? list)
+    internal bool TryGetList([NotNullWhen(true)] out List<ObservableGroup<TKey, TElement>>? list)
     {
-        list = Items as List<ObservableGroup<TKey, TValue>>;
+        list = Items as List<ObservableGroup<TKey, TElement>>;
 
         return list is not null;
     }

CommunityToolkit.Mvvm/Collections/ObservableGroup{TKey,TValue}.cs renamed to CommunityToolkit.Mvvm/Collections/ObservableGroup{TKey,TElement}.cs

@@ -15,9 +15,9 @@ namespace CommunityToolkit.Mvvm.Collections;
 /// It associates a <see cref="Key"/> to an <see cref="ObservableCollection{T}"/>.
 /// </summary>
 /// <typeparam name="TKey">The type of the group key.</typeparam>
-/// <typeparam name="TValue">The type of the items in the collection.</typeparam>
+/// <typeparam name="TElement">The type of elements in the group.</typeparam>
 [DebuggerDisplay("Key = {Key}, Count = {Count}")]
-public class ObservableGroup<TKey, TValue> : ObservableCollection<TValue>, IReadOnlyObservableGroup<TKey, TValue>
+public class ObservableGroup<TKey, TElement> : ObservableCollection<TElement>, IReadOnlyObservableGroup<TKey, TElement>
     where TKey : notnull
 {
     /// <summary>
@@ -33,7 +33,7 @@ public ObservableGroup(TKey key)
     /// Initializes a new instance of the <see cref="ObservableGroup{TKey, TValue}"/> class.
     /// </summary>
     /// <param name="grouping">The grouping to fill the group.</param>
-    public ObservableGroup(IGrouping<TKey, TValue> grouping)
+    public ObservableGroup(IGrouping<TKey, TElement> grouping)
         : base(grouping)
     {
         this.key = grouping.Key;
@@ -44,7 +44,7 @@ public ObservableGroup(IGrouping<TKey, TValue> grouping)
     /// </summary>
     /// <param name="key">The key for the group.</param>
     /// <param name="collection">The initial collection of data to add to the group.</param>
-    public ObservableGroup(TKey key, IEnumerable<TValue> collection)
+    public ObservableGroup(TKey key, IEnumerable<TElement> collection)
         : base(collection)
     {
         this.key = key;

CommunityToolkit.Mvvm/Collections/ReadOnlyObservableGroupedCollection{TKey,TValue}.cs renamed to CommunityToolkit.Mvvm/Collections/ReadOnlyObservableGroupedCollection{TKey,TElement}.cs

@@ -13,17 +13,17 @@ namespace CommunityToolkit.Mvvm.Collections;
 /// <summary>
 /// A read-only list of groups.
 /// </summary>
-/// <typeparam name="TKey">The type of the group key.</typeparam>
-/// <typeparam name = "TValue" > The type of the items in the collection.</typeparam>
-public sealed class ReadOnlyObservableGroupedCollection<TKey, TValue> : ReadOnlyObservableCollection<ReadOnlyObservableGroup<TKey, TValue>>
+/// <typeparam name="TKey">The type of the group keys.</typeparam>
+/// <typeparam name="TElement">The type of elements in the collection.</typeparam>
+public sealed class ReadOnlyObservableGroupedCollection<TKey, TElement> : ReadOnlyObservableCollection<ReadOnlyObservableGroup<TKey, TElement>>
     where TKey : notnull
 {
     /// <summary>
     /// Initializes a new instance of the <see cref="ReadOnlyObservableGroupedCollection{TKey, TValue}"/> class.
     /// </summary>
     /// <param name="collection">The source collection to wrap.</param>
-    public ReadOnlyObservableGroupedCollection(ObservableCollection<ObservableGroup<TKey, TValue>> collection)
-        : base(new ObservableCollection<ReadOnlyObservableGroup<TKey, TValue>>(collection.Select(static g => new ReadOnlyObservableGroup<TKey, TValue>(g))))
+    public ReadOnlyObservableGroupedCollection(ObservableCollection<ObservableGroup<TKey, TElement>> collection)
+        : base(new ObservableCollection<ReadOnlyObservableGroup<TKey, TElement>>(collection.Select(static g => new ReadOnlyObservableGroup<TKey, TElement>(g))))
     {
         collection.CollectionChanged += OnSourceCollectionChanged;
     }
@@ -32,7 +32,7 @@ public ReadOnlyObservableGroupedCollection(ObservableCollection<ObservableGroup<
     /// Initializes a new instance of the <see cref="ReadOnlyObservableGroupedCollection{TKey, TValue}"/> class.
     /// </summary>
     /// <param name="collection">The source collection to wrap.</param>
-    public ReadOnlyObservableGroupedCollection(ObservableCollection<ReadOnlyObservableGroup<TKey, TValue>> collection)
+    public ReadOnlyObservableGroupedCollection(ObservableCollection<ReadOnlyObservableGroup<TKey, TElement>> collection)
         : base(collection)
     {
     }
@@ -59,17 +59,17 @@ static void ThrowNotSupportedExceptionForRangeOperation()
         }
 
         // The inner Items list is ObservableCollection<ReadOnlyObservableGroup<TKey, TValue>>, so doing a direct cast here will always succeed
-        ObservableCollection<ReadOnlyObservableGroup<TKey, TValue>> items = (ObservableCollection<ReadOnlyObservableGroup<TKey, TValue>>)Items;
+        ObservableCollection<ReadOnlyObservableGroup<TKey, TElement>> items = (ObservableCollection<ReadOnlyObservableGroup<TKey, TElement>>)Items;
 
         switch (e.Action)
         {
             // Insert a single item for an "Add" operation, fail if multiple items are added
             case NotifyCollectionChangedAction.Add:
                 if (e.NewItems!.Count == 1)
                 {
-                    ObservableGroup<TKey, TValue> newItem = (ObservableGroup<TKey, TValue>)e.NewItems![0]!;
+                    ObservableGroup<TKey, TElement> newItem = (ObservableGroup<TKey, TElement>)e.NewItems![0]!;
 
-                    items.Insert(e.NewStartingIndex, new ReadOnlyObservableGroup<TKey, TValue>(newItem));
+                    items.Insert(e.NewStartingIndex, new ReadOnlyObservableGroup<TKey, TElement>(newItem));
                 }
                 else if (e.NewItems!.Count > 1)
                 {
@@ -95,9 +95,9 @@ static void ThrowNotSupportedExceptionForRangeOperation()
             case NotifyCollectionChangedAction.Replace:
                 if (e.OldItems!.Count == 1 && e.NewItems!.Count == 1)
                 {
-                    ObservableGroup<TKey, TValue> replacedItem = (ObservableGroup<TKey, TValue>)e.NewItems![0]!;
+                    ObservableGroup<TKey, TElement> replacedItem = (ObservableGroup<TKey, TElement>)e.NewItems![0]!;
 
-                    items[e.OldStartingIndex] = new ReadOnlyObservableGroup<TKey, TValue>(replacedItem);
+                    items[e.OldStartingIndex] = new ReadOnlyObservableGroup<TKey, TElement>(replacedItem);
                 }
                 else if (e.OldItems!.Count > 1 || e.NewItems!.Count > 1)
                 {

CommunityToolkit.Mvvm/Collections/ReadOnlyObservableGroup{TKey,TValue}.cs renamed to CommunityToolkit.Mvvm/Collections/ReadOnlyObservableGroup{TKey,TElement}.cs

@@ -10,16 +10,16 @@ namespace CommunityToolkit.Mvvm.Collections;
 /// A read-only observable group. It associates a <see cref="Key"/> to a <see cref="ReadOnlyObservableCollection{T}"/>.
 /// </summary>
 /// <typeparam name="TKey">The type of the group key.</typeparam>
-/// <typeparam name="TValue">The type of the items in the collection.</typeparam>
-public sealed class ReadOnlyObservableGroup<TKey, TValue> : ReadOnlyObservableCollection<TValue>, IReadOnlyObservableGroup<TKey, TValue>
+/// <typeparam name="TElement">The type of elements in the group.</typeparam>
+public sealed class ReadOnlyObservableGroup<TKey, TElement> : ReadOnlyObservableCollection<TElement>, IReadOnlyObservableGroup<TKey, TElement>
     where TKey : notnull
 {
     /// <summary>
     /// Initializes a new instance of the <see cref="ReadOnlyObservableGroup{TKey, TValue}"/> class.
     /// </summary>
     /// <param name="key">The key of the group.</param>
     /// <param name="collection">The collection of items to add in the group.</param>
-    public ReadOnlyObservableGroup(TKey key, ObservableCollection<TValue> collection)
+    public ReadOnlyObservableGroup(TKey key, ObservableCollection<TElement> collection)
         : base(collection)
     {
         Key = key;
@@ -29,7 +29,7 @@ public ReadOnlyObservableGroup(TKey key, ObservableCollection<TValue> collection
     /// Initializes a new instance of the <see cref="ReadOnlyObservableGroup{TKey, TValue}"/> class.
     /// </summary>
     /// <param name="group">The <see cref="ObservableGroup{TKey, TValue}"/> to wrap.</param>
-    public ReadOnlyObservableGroup(ObservableGroup<TKey, TValue> group)
+    public ReadOnlyObservableGroup(ObservableGroup<TKey, TElement> group)
         : base(group)
     {
         Key = group.Key;