From 66883f7f31ec6b4367d456c461df111a26d805ab Mon Sep 17 00:00:00 2001 From: Bret Johnson Date: Tue, 19 Apr 2022 12:00:30 -0400 Subject: Remove UpdateExpansions Remove the call to UpdateExpansions after ReloadData in response to OnPropertiesChanged (e.g. called when selection changes). This doesn't seem necessary, as when views are recreated their expansion status is set correctly originally and AppKit remembers expansion state after. Further, this fixes AB#1521131, where the Name and Type rows are sometimes blank - apparently expanding manually, and doing it synchronously like that, causes the NSOutlineView to get confused and not show the top row sometimes. --- Xamarin.PropertyEditing.Mac/PropertyList.cs | 2 -- 1 file changed, 2 deletions(-) diff --git a/Xamarin.PropertyEditing.Mac/PropertyList.cs b/Xamarin.PropertyEditing.Mac/PropertyList.cs index 027b21f..7e0d9aa 100644 --- a/Xamarin.PropertyEditing.Mac/PropertyList.cs +++ b/Xamarin.PropertyEditing.Mac/PropertyList.cs @@ -153,8 +153,6 @@ namespace Xamarin.PropertyEditing.Mac private void OnPropertiesChanged (object sender, EventArgs e) { this.propertyTable.ReloadData (); - - ((PropertyTableDelegate)this.propertyTable.Delegate).UpdateExpansions (this.propertyTable); } private void UpdateResourceProvider () -- cgit v1.2.3