From f3fdd86a0324a04c920d1a3af9a966f38051d51b Mon Sep 17 00:00:00 2001
From: Antony Male <antony.male@gmail.com>
Date: Wed, 12 Mar 2014 12:13:15 +0000
Subject: [PATCH] Document BindableCollection

---
 Stylet/BindableCollection.cs | 37 ++++++++++++++++++++++++++++++------
 1 file changed, 31 insertions(+), 6 deletions(-)

diff --git a/Stylet/BindableCollection.cs b/Stylet/BindableCollection.cs
index b0923da..8f08f59 100644
--- a/Stylet/BindableCollection.cs
+++ b/Stylet/BindableCollection.cs
@@ -10,25 +10,39 @@ using System.Threading.Tasks;
 
 namespace Stylet
 {
+    /// <summary>
+    /// Represents a collection which is observasble
+    /// </summary>
+    /// <typeparam name="T"></typeparam>
     public interface IObservableCollection<T> : IList<T>, INotifyPropertyChanged, INotifyCollectionChanged
     {
+        /// <summary>
+        /// Add a range of items
+        /// </summary>
+        /// <param name="items">Items to add</param>
         void AddRange(IEnumerable<T> items);
+
+        /// <summary>
+        /// Remove a range of items
+        /// </summary>
+        /// <param name="items">Items to remove</param>
         void RemoveRange(IEnumerable<T> items);
     }
 
+    /// <summary>
+    /// ObservableCollection subclass which supports AddRange and RemoveRange
+    /// </summary>
+    /// <typeparam name="T"></typeparam>
     public class BindableCollection<T> : ObservableCollection<T>, IObservableCollection<T>
     {
+        /// <summary>
+        ///  We have to disable notifications when adding individual elements in the AddRange and RemoveRange implementations
+        /// </summary>
         private bool isNotifying = true;
 
         public BindableCollection() : base() { }
         public BindableCollection(IEnumerable<T> collection) : base(collection) { }
 
-        protected void NotifyOfPropertyChange([CallerMemberName] string propertyName = "")
-        {
-            if (this.isNotifying)
-                this.OnPropertyChanged(new PropertyChangedEventArgs(propertyName));
-        }
-
         protected override void OnPropertyChanged(PropertyChangedEventArgs e)
         {
             if (this.isNotifying)
@@ -41,6 +55,10 @@ namespace Stylet
                 base.OnCollectionChanged(e);
         }
 
+        /// <summary>
+        /// Add a range of items
+        /// </summary>
+        /// <param name="items">Items to add</param>
         public virtual void AddRange(IEnumerable<T> items)
         {
            var previousNotificationSetting = this.isNotifying;
@@ -57,6 +75,10 @@ namespace Stylet
             this.OnCollectionChanged(new NotifyCollectionChangedEventArgs(NotifyCollectionChangedAction.Add, items.ToList()));
         }
 
+        /// <summary>
+        /// Remove a range of items
+        /// </summary>
+        /// <param name="items">Items to remove</param>
         public virtual void RemoveRange(IEnumerable<T> items)
         {
             var previousNotificationSetting = this.isNotifying;
@@ -75,6 +97,9 @@ namespace Stylet
             this.OnCollectionChanged(new NotifyCollectionChangedEventArgs(NotifyCollectionChangedAction.Remove, items.ToList()));
         }
 
+        /// <summary>
+        /// Raise a change notification indicating that all bindings should be refreshed
+        /// </summary>
         public void Refresh()
         {
             this.OnPropertyChanged(new PropertyChangedEventArgs("Count"));