Goal: support standard ObservableCollection instead of custom ObservableList to avoid tight coupling with UnityWeld.

Discussion:
Real-Serious-Games#36

Done:
- Remove support of .net3.5 (it's deprecated in Unity since 2018 version)
- Remove custom ObservableList, INotifyCollectionChanged, NotifyCollectionChangedEventArgs
- Replace refs to ObservableList with ObservableCollection

Author: TimGameDev

Unity-Weld.nuspec

@@ -16,8 +16,6 @@
     <tags>C# Unity Unity3D UI MVVM data binding databinding</tags>
   </metadata>
   <files>
-    <file src="UnityWeld/bin/Release/net35/UnityWeld.dll" target="lib\net35"/>
-    <file src="UnityWeld_Editor/bin/Release/net35/UnityWeld_Editor.dll" target="lib\net35"/>
     <file src="UnityWeld/bin/Release/netstandard2.0/UnityWeld.dll" target="lib\netstandard2.0"/>
     <file src="UnityWeld_Editor/bin/Release/netstandard2.0/UnityWeld_Editor.dll" target="lib\netstandard2.0"/>
   </files>

UnityWeld/Binding/BoundObservableList.cs

@@ -1,5 +1,7 @@
-using System;
+using System;
 using System.Collections.Generic;
+using System.Collections.ObjectModel;
+using System.Collections.Specialized;
 using System.Linq;
 
 
@@ -8,12 +10,12 @@ namespace UnityWeld.Binding
     /// <summary>
     /// An observable list that is bound to source list.
     /// </summary>
-    public class BoundObservableList<DestT, SourceT> : ObservableList<DestT>, IDisposable
+    public class BoundObservableList<DestT, SourceT> : ObservableCollection<DestT>, IDisposable
     {
         /// <summary>
         /// The source list.
         /// </summary>
-        private readonly ObservableList<SourceT> source;
+        private readonly ObservableCollection<SourceT> source;
 
         /// <summary>
         /// Function that maps source items to dest items.
@@ -43,7 +45,7 @@ public class BoundObservableList<DestT, SourceT> : ObservableList<DestT>, IDispo
 
         private bool disposed;
 
-        public BoundObservableList(ObservableList<SourceT> source, Func<SourceT, DestT> itemMap) :
+        public BoundObservableList(ObservableCollection<SourceT> source, Func<SourceT, DestT> itemMap) :
             base(source.Select(itemMap))
         {
             this.itemMap = itemMap;
@@ -55,7 +57,7 @@ public BoundObservableList(ObservableList<SourceT> source, Func<SourceT, DestT>
             cache = new List<DestT>(this);
         }
 
-        public BoundObservableList(ObservableList<SourceT> source, Func<SourceT, DestT> itemMap, Action<DestT> added, Action<DestT> removed) :
+        public BoundObservableList(ObservableCollection<SourceT> source, Func<SourceT, DestT> itemMap, Action<DestT> added, Action<DestT> removed) :
             base(source.Select(itemMap))
         {
             if (added == null)
@@ -82,7 +84,7 @@ public BoundObservableList(ObservableList<SourceT> source, Func<SourceT, DestT>
             cache = new List<DestT>(this);
         }
 
-        public BoundObservableList(ObservableList<SourceT> source, Func<SourceT, DestT> itemMap, Action<DestT> added, Action<DestT> removed, Action changed) :
+        public BoundObservableList(ObservableCollection<SourceT> source, Func<SourceT, DestT> itemMap, Action<DestT> added, Action<DestT> removed, Action changed) :
             base(source.Select(itemMap))
         {
             if (added == null)

UnityWeld/Binding/CollectionBinding.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections;
+using System.Collections.Specialized;
 using System.Linq;
 using UnityEngine;
 using UnityWeld.Binding.Exceptions;