Added workaround for node view with list bug

Author: Antoine Lelievre

Editor/Views/BaseNodeView.cs

@@ -234,11 +234,6 @@ void UpdateTitle()
 			title = (nodeTarget.GetCustomName() == null) ? nodeTarget.GetType().Name : nodeTarget.GetCustomName();
 		}
 
-		public void UpdateNodeSerializedPropertyBindings()
-		{
-
-		}
-
 		void InitializeSettings()
 		{
 			// Initialize settings button:
@@ -270,6 +265,21 @@ void OnGeometryChanged(GeometryChangedEvent evt)
 				settingsContainer.style.left = settingsButtonLayout.xMin - layout.width + 20f;
 			}
 		}
+
+		// Workaround for bug in GraphView that makes the node selection border way too big
+		VisualElement selectionBorder, nodeBorder;
+		internal void EnableSyncSelectionBorderHeight()
+		{
+			if (selectionBorder == null || nodeBorder == null)
+			{
+				selectionBorder = this.Q("selection-border");
+				nodeBorder = this.Q("node-border");
+
+				schedule.Execute(() => {
+					selectionBorder.style.height = nodeBorder.localBound.height;
+				}).Every(17);
+			}
+		}
 
 		void CreateSettingButton()
 		{
@@ -681,7 +691,6 @@ protected virtual void DrawDefaultInspector(bool fromInspector = false)
 					continue;
 				}
 
-
 				//skip if the field is an input/output and not marked as SerializedField
 				bool hasInputAttribute         = field.GetCustomAttribute(typeof(InputAttribute)) != null;
 				bool hasInputOrOutputAttribute = hasInputAttribute || field.GetCustomAttribute(typeof(OutputAttribute)) != null;
@@ -852,6 +861,9 @@ protected VisualElement AddControlField(FieldInfo field, string label = null, bo
 				element.AddToClassList("DrawerField_2020_3");
 #endif
 
+			if (typeof(IList).IsAssignableFrom(field.FieldType))
+				EnableSyncSelectionBorderHeight();
+
 			element.RegisterValueChangeCallback(e => {
 				UpdateFieldVisibility(field.Name, field.GetValue(nodeTarget));
 				valueChangedCallback?.Invoke();