Bug in RemoveAt function

Editor/UnityDictionaryDrawer.cs

@@ -0,0 +1,42 @@
+using UnityEditor;
+using UnityEngine;
+
+public class UnityDictionaryDrawer<TKey, TValue> : PropertyDrawer
+{
+    override public void OnGUI(Rect position, SerializedProperty property, GUIContent label)
+    {
+        var propTitle = EditorGUI.BeginProperty(position, label, property);
+
+        // PropertyDrawers don't support GUILayout so we have to construct our own rectangles
+        var foldoutRect = new Rect(position.left, position.top, position.width, base.GetPropertyHeight(property, label));
+        bool wasExpanded = property.isExpanded;
+        property.isExpanded = EditorGUI.Foldout(foldoutRect, property.isExpanded, propTitle);
+
+        // Don't use the latest isExpanded value because if it's changed then we're the wrong height
+        if(wasExpanded)
+        {
+
+        }
+
+        EditorGUI.EndProperty();
+    }
+
+    public override float GetPropertyHeight(SerializedProperty property, GUIContent label)
+    {
+        var total = base.GetPropertyHeight(property, label);
+        if(property.isExpanded)
+        {
+            total += base.GetPropertyHeight(property, label) * property.FindPropertyRelative("_keys").arraySize;
+        }
+
+        return total;
+    }
+}
+
+//TODO: figure out how to make it so you dont need to have one per dummy subclass.
+
+[CustomPropertyDrawer(typeof(UnityDictionaryIntString))]
+public class UnityDictionaryDrawerIntString : UnityDictionaryDrawer<int, string>
+{
+
+}

ReadOnlyListWrapper.cs

@@ -0,0 +1,81 @@
+using System;
+using System.Collections;
+using System.Collections.Generic;
+
+internal class ReadOnlyListWrapper<T> : ICollection<T>, ICollection
+{
+    private readonly List<T> _wrappedList;
+
+    public ReadOnlyListWrapper(List<T> listToWrap)
+    {
+        _wrappedList = listToWrap;
+    }
+
+    #region Implementation of IEnumerable
+
+    public IEnumerator<T> GetEnumerator()
+    {
+        return _wrappedList.GetEnumerator();
+    }
+
+    IEnumerator IEnumerable.GetEnumerator()
+    {
+        return GetEnumerator();
+    }
+
+    #endregion
+
+    #region Implementation of ICollection<T>
+
+    public void Add(T item)
+    {
+        throw new NotSupportedException("The list is read-only.");
+    }
+
+    public void Clear()
+    {
+        throw new NotSupportedException("The list is read-only.");
+    }
+
+    public bool Contains(T item)
+    {
+        return _wrappedList.Contains(item);
+    }
+
+    public void CopyTo(T[] array, int arrayIndex)
+    {
+        _wrappedList.CopyTo(array, arrayIndex);
+    }
+
+    public bool Remove(T item)
+    {
+        throw new NotSupportedException("The list is read-only.");
+    }
+
+    public void CopyTo(Array array, int index)
+    {
+        ((ICollection)_wrappedList).CopyTo(array, index);
+    }
+
+    public int Count
+    {
+        get { return _wrappedList.Count; }
+    }
+
+    public object SyncRoot
+    {
+        get { return ((ICollection) _wrappedList).SyncRoot; }
+    }
+
+    public bool IsSynchronized
+    {
+        get { return ((ICollection) _wrappedList).IsSynchronized; }
+    }
+
+    public bool IsReadOnly
+    {
+        get { return true; }
+    }
+
+    #endregion
+}