Browse Source

Removed unused itemTemplate parameter.

Steven Kirk 10 years ago
parent
commit
7cc5ee5dcc

+ 1 - 3
src/Perspex.Controls/Generators/IItemContainerGenerator.cs

@@ -25,12 +25,10 @@ namespace Perspex.Controls.Generators
         /// The index of the first item of the data in the containing collection.
         /// </param>
         /// <param name="items">The items.</param>
-        /// <param name="itemTemplate">An optional item template.</param>
         /// <returns>The created controls.</returns>
         IList<IControl> CreateContainers(
             int startingIndex,
-            IEnumerable items,
-            IDataTemplate itemTemplate);
+            IEnumerable items);
 
         /// <summary>
         /// Removes a set of created containers from the index and returns the removed controls.

+ 3 - 12
src/Perspex.Controls/Generators/ItemContainerGenerator.cs

@@ -45,12 +45,10 @@ namespace Perspex.Controls.Generators
         /// The index of the first item of the data in the containing collection.
         /// </param>
         /// <param name="items">The items.</param>
-        /// <param name="itemTemplate">An optional item template.</param>
         /// <returns>The created container controls.</returns>
         public IList<IControl> CreateContainers(
             int startingIndex,
-            IEnumerable items,
-            IDataTemplate itemTemplate)
+            IEnumerable items)
         {
             Contract.Requires<ArgumentNullException>(items != null);
 
@@ -59,7 +57,7 @@ namespace Perspex.Controls.Generators
 
             foreach (var item in items)
             {
-                IControl container = CreateContainer(item, itemTemplate);
+                IControl container = CreateContainer(item);
                 result.Add(container);
             }
 
@@ -141,20 +139,13 @@ namespace Perspex.Controls.Generators
         /// Creates the container for an item.
         /// </summary>
         /// <param name="item">The item.</param>
-        /// <param name="itemTemplate">An optional item template.</param>
         /// <returns>The created container control.</returns>
-        protected virtual IControl CreateContainer(object item, IDataTemplate itemTemplate)
+        protected virtual IControl CreateContainer(object item)
         {
             if (item == null)
             {
                 return null;
             }
-            else if (itemTemplate != null && itemTemplate.Match(item))
-            {
-                var result = itemTemplate.Build(item);
-                result.DataContext = item;
-                return result;
-            }
             else
             {
                 return Owner.MaterializeDataTemplate(item);

+ 1 - 1
src/Perspex.Controls/Generators/ItemContainerGenerator`1.cs

@@ -21,7 +21,7 @@ namespace Perspex.Controls.Generators
         }
 
         /// <inheritdoc/>
-        protected override IControl CreateContainer(object item, IDataTemplate itemTemplate)
+        protected override IControl CreateContainer(object item)
         {
             T result = item as T;
 

+ 3 - 5
src/Perspex.Controls/Generators/TreeItemContainerGenerator.cs

@@ -46,9 +46,8 @@ namespace Perspex.Controls.Generators
         /// The index of the first item of the data in the containing collection.
         /// </param>
         /// <param name="items">The items.</param>
-        /// <param name="itemTemplate">An optional item template.</param>
         /// <returns>The created container controls.</returns>
-        public IList<IControl> CreateContainers(int startingIndex, IEnumerable items, IDataTemplate itemTemplate)
+        public IList<IControl> CreateContainers(int startingIndex, IEnumerable items)
         {
             Contract.Requires<ArgumentNullException>(items != null);
 
@@ -57,7 +56,7 @@ namespace Perspex.Controls.Generators
 
             foreach (var item in items)
             {
-                var container = CreateContainer(item, itemTemplate);
+                var container = CreateContainer(item);
                 _containers.Add(item, container);
                 result.Add(container);
             }
@@ -156,9 +155,8 @@ namespace Perspex.Controls.Generators
         /// Creates the container for an item.
         /// </summary>
         /// <param name="item">The item.</param>
-        /// <param name="itemTemplate">An optional item template.</param>
         /// <returns>The created container control.</returns>
-        protected virtual T CreateContainer(object item, IDataTemplate itemTemplate)
+        protected virtual T CreateContainer(object item)
         {
             T result = item as T;
 

+ 1 - 1
src/Perspex.Controls/Presenters/DeckPresenter.cs

@@ -181,7 +181,7 @@ namespace Perspex.Controls.Presenters
             if (toIndex != -1)
             {
                 var item = Items.Cast<object>().ElementAt(toIndex);
-                to = generator.CreateContainers(toIndex, new[] { item }, null).FirstOrDefault();
+                to = generator.CreateContainers(toIndex, new[] { item }).FirstOrDefault();
 
                 if (to != null)
                 {

+ 2 - 2
src/Perspex.Controls/Presenters/ItemsPresenter.cs

@@ -172,7 +172,7 @@ namespace Perspex.Controls.Presenters
             if (items != null)
             {
                 Panel.Children.AddRange(
-                    ItemContainerGenerator.CreateContainers(0, Items, null));
+                    ItemContainerGenerator.CreateContainers(0, Items));
 
                 INotifyCollectionChanged incc = items as INotifyCollectionChanged;
 
@@ -229,7 +229,7 @@ namespace Perspex.Controls.Presenters
                 {
                     case NotifyCollectionChangedAction.Add:
                         Panel.Children.AddRange(
-                            generator.CreateContainers(e.NewStartingIndex, e.NewItems, null));
+                            generator.CreateContainers(e.NewStartingIndex, e.NewItems));
                         break;
 
                     case NotifyCollectionChangedAction.Remove: