Tomasz Chabora
Tomasz Chabora
Why is this editor-only? This makes the newly added `focus` property useless at runtime. EDIT: No, it's entirely useless, because `set_draw_focus_border()` is not exposed. If there is a good reason...
If you want it editor-only, it should be a new type that isn't exposed. Adding things to ScrollContainer that can't be used is not the right solution.
Tested and it seems to work correctly. ScrollContainer does not need to be focused to display the border, so I think the focus change to the inspector is not needed....
For Vector4, #80893 provides an alternative layout. Maybe it's worth it to provide option for every vector type separately? It's weird to have `vector_types_display_style`, and a separate option for Vector2....
Your PR already renames the existing settings, adding a new one wouldn't be much different.
Yes, the current master is a bit broken. I'll fix the errors before next release. If you want to get the stable version from GitHub, use this commit 17e1fed0ec7cd1a9bffbe58301edbab49b02b16e
Makes sense. Although the master is usually stable. I just started some big refactor and it's dragging out.
This would possibly supersede #266
Ah right. Well, at least it would make it possible to easily resolve that proposal, as the required functionality already exists.
He was probably referring to the texture region editor, which was since then moved to a dedicated dialog opened from the inspector.