GridView's Refresh() is not working properly

[bai-chi-xing]

1687532661812.mp4

The above is a demo video, and the following is the code. If the commented-out code is added, the UI will behave normally, but I believe this is not the proper approach.

public class NewBehaviourScript : MonoBehaviour
{
    public GridView gridView;
    public ListView listView;
    public UIBlock2D btn_add;
    public UIBlock2D btn_remove;
    public List<int> ints = new List<int>() { 0, 1 };

    void OnEnable()
    {
        btn_add.AddGestureHandler<Gesture.OnClick>(click =>
        {
            ints.Add(ints.Count);
            //gridView.SetDataSource(ints);
            gridView.Refresh();
            listView.Refresh();
        });
        btn_remove.AddGestureHandler<Gesture.OnClick>(click =>
        {
            ints.RemoveAt(ints.Count - 1);
            //gridView.SetDataSource(ints);
            gridView.Refresh();
            listView.Refresh();
        });
        gridView.AddDataBinder<int, ButtonVisuals>((bind, visuals, index) => {});
        listView.AddDataBinder<int, ButtonVisuals>((bind, visuals, index) => {});
        gridView.SetDataSource(ints);
        listView.SetDataSource(ints);
    }
}

[adgray1]

Thank you for the video and test code! Really appreciate it.

While we do see a bug with GridView.Refresh() using the code you provided, it seems different from the issue shown in your video.

So we have a few follow up questions to help us better understand if there's more than one thing we need to go fix:

1. What version of Nova are you using in the video?
2. Would you mind sharing a screenshot of your GridView editor configuration along with the GridView's UIBlock size and auto layout configurations?

[bai-chi-xing]

This is a screenshot. Additionally, I have an asset package that depends on TextMeshPro and Nova 2.0.0, and the Unity version is 2022.3.0f1c1.
BUG.zip

[adgray1]

We're able to reproduce the issue with the example in the unitypackage you attached, so thank you for that along with the additional details!

As suspected after our initial testing, there does appear to be more than one part of Refresh() not working entirely as expected, but we'll be sure to follow up here once we have the fix(es) available.

[adgray1]

The fixes for these refresh issues have been published in Nova v2.0.1!