RefreshItems using same logic as ActionMapsView

tspiller · tspiller · commit 40b6dce8930b · 2024-12-09T15:42:02.000Z
diff --git a/Packages/com.unity.inputsystem/InputSystem/Editor/UITKAssetEditor/Views/ActionsTreeView.cs b/Packages/com.unity.inputsystem/InputSystem/Editor/UITKAssetEditor/Views/ActionsTreeView.cs
@@ -204,7 +204,13 @@ public override void RedrawUI(ViewState viewState)
         {
             m_ActionsTreeView.Clear();
             m_ActionsTreeView.SetRootItems(viewState.treeViewData);
+            // UI toolkit doesn't behave the same on 6000.0 way when refreshing items
+            // On previous versions, we need to call Rebuild() to refresh the items since refreshItems() is less predicatable
+#if UNITY_6000_0_OR_NEWER
+            m_ActionsTreeView.RefreshItems();
+#else
             m_ActionsTreeView.Rebuild();
+#endif
             if (viewState.newElementID != -1)
             {
                 m_ActionsTreeView.SetSelectionById(viewState.newElementID);

Original file line number	Diff line number	Diff line change
`@@ -204,7 +204,13 @@ public override void RedrawUI(ViewState viewState)`
`204`	`204`	`{`
`205`	`205`	`m_ActionsTreeView.Clear();`
`206`	`206`	`m_ActionsTreeView.SetRootItems(viewState.treeViewData);`
	`207`	`+ // UI toolkit doesn't behave the same on 6000.0 way when refreshing items`
	`208`	`+ // On previous versions, we need to call Rebuild() to refresh the items since refreshItems() is less predicatable`
	`209`	`+#if UNITY_6000_0_OR_NEWER`
	`210`	`+ m_ActionsTreeView.RefreshItems();`
	`211`	`+#else`
`207`	`212`	`m_ActionsTreeView.Rebuild();`
	`213`	`+#endif`
`208`	`214`	`if (viewState.newElementID != -1)`
`209`	`215`	`{`
`210`	`216`	`m_ActionsTreeView.SetSelectionById(viewState.newElementID);`