Unable to ctrl+z to undo when editing the field.

The editor is unable to perform undo(ctrl+z) when editing the field. (e.g. PropertyField, TextField, IntegerField )

Is this by design or I did something wrong?

Kindly please advice. Thanks.

Code (CSharp):

1. using UnityEngine;
2.  
3. public class ItemCollection : MonoBehaviour {
4.  
5.     [SerializeField]
6.     private int quantity = 0;
7.  
8.     void Start () {
9.         Debug.LogFormat("Quantity : {0}", quantity);
10.     }
11. }
12.  

Code (CSharp):

1. using UnityEngine;
2. using UnityEditor;
3. using UnityEngine.UIElements;
4. using UnityEditor.UIElements;
5.  
6. [CanEditMultipleObjects]
7. [CustomEditor(typeof(ItemCollection))]
8. public class ItemCollectionEditor : Editor {
9.  
10.     private SerializedProperty quantityProp;
11.  
12.     void OnEnable () {
13.         quantityProp = serializedObject.FindProperty("quantity");
14.     }
15.  
16.     // Work fine when using IMGUI.
17.     /*
18.     public override void OnInspectorGUI () {
19.         serializedObject.Update();
20.  
21.         EditorGUILayout.PropertyField(quantityProp);
22.  
23.         serializedObject.ApplyModifiedProperties();
24.     }
25.     */
26.  
27.     // Unable to perform ctrl+z to undo when editing the field using UIElement.
28.     public override VisualElement CreateInspectorGUI () {
29.         var root = new VisualElement();
30.  
31.         var quantityField = new PropertyField(quantityProp);
32.         quantityField.Bind(serializedObject);
33.         root.Add(quantityField);
34.  
35.         return root;
36.     }
37.  
38. }
39.  

 

Hi @hugobd,

Just submit the bug report. Thanks.

 

Any update on this?

 

any update on this? would love to not have to tell people you can't undo this, can't undo that.

 

is there any way to setup undo/redo on normal UITK fields like the checkbox, intSlider, etc?

 

so to make an editor that previews things or is used to tweak things but not bound to a particular object then we have to figure out how to add undo/redo to them manually?

would making a "Scriptable Object" be sufficient to get the undo/redo logic to be inherent?