[Bug] ReorderableList does not respect indentation

[p-monteiro]

Reorderable Lists fill their parent container instead of respecting the parent indentation level.

In the image example, both lists should be placed at the same level as "World Metadata" as they are declared inside of the "World" class

[arimger]

Hi! Is this an EditorWindow displaying a custom Editor, or is it the standard InspectorWindow?

[p-monteiro]

It is the standard editor for a scriptable object. But I reckon it is wrong for the EditorWindow as well as a standard Inspector.

Maybe try to use EditorGUI.IndentedRect() to get the proper rects.

[arimger]

The thing is that everything works fine on my end. Internally, ReorderableList is in a zero-indent scope, but this is on purpose to draw all related controls properly. The initial position is fixed by a layout group.

Can you give me a test code example with a class that has similar issues? Maybe I'm missing something.

[p-monteiro]

As you can see, all lists are indented at 0 and not respecting their level, which prevents creating more complex editors since visual hierarchy is important.

Additionally, it also does not support other attributes such as [Header()].

Test code:

using System;
using System.Collections.Generic;
using UnityEngine;

public class TestReorderableList : MonoBehaviour
{
    [Serializable]
    public class SecondOrder
    {
        public int secondOrderField;

        [Header("Test"), ReorderableList]
        public List<int> secondOrderList = new();

        public ThirdOrder thirdOrder = new();
    }

    [Serializable]
    public class ThirdOrder
    {
        public int thirdOrderField;

        [Header("Test"), ReorderableList]
        public List<int> thirdOrderList = new();
    }

    public int firstOrderField;

    [Header("Test"), ReorderableList]
    public List<int> firstOrderList = new();

    public SecondOrder secondOrder = new();
}

[arimger]

Fixed in #144.