Skip to content

Commit c687272

Browse files
authored
Improved material renderer inspector (#590)
* Added a separator to the material renderer to separate visibility settings and materials * Added formatted name for material entries
1 parent fe15930 commit c687272

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

Sources/Overload/OvCore/src/OvCore/ECS/Components/CMaterialRenderer.cpp

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,10 @@
1818
#include <OvUI/Widgets/Buttons/ButtonSmall.h>
1919
#include <OvUI/Plugins/DDTarget.h>
2020
#include <OvUI/Widgets/InputFields/InputInt.h>
21+
#include <OvUI/Widgets/Layout/Dummy.h>
2122
#include <OvUI/Widgets/Layout/Group.h>
2223
#include <OvUI/Widgets/Texts/TextColored.h>
24+
#include <OvUI/Widgets/Visual/Separator.h>
2325

2426
OvCore::ECS::Components::CMaterialRenderer::CMaterialRenderer(ECS::Actor & p_owner) : AComponent(p_owner)
2527
{
@@ -201,8 +203,13 @@ void OvCore::ECS::Components::CMaterialRenderer::OnInspector(OvUI::Internal::Wid
201203
drawVisibilityToggle("Reflection", REFLECTION);
202204
drawVisibilityToggle("Shadow", SHADOW);
203205

206+
p_root.CreateWidget<OvUI::Widgets::Visual::Separator>();
207+
p_root.CreateWidget<OvUI::Widgets::Layout::Dummy>(); // Necessary to fill the "value" column
208+
204209
for (uint8_t i = 0; i < m_materials.size(); ++i)
210+
{
205211
m_materialFields[i] = CustomMaterialDrawer(p_root, "Material", m_materials[i]);
212+
}
206213

207214
UpdateMaterialList();
208215
}
@@ -230,7 +237,8 @@ void OvCore::ECS::Components::CMaterialRenderer::UpdateMaterialList()
230237
m_materialFields[i][0]->enabled = enabled;
231238
m_materialFields[i][1]->enabled = enabled;
232239
m_materialFields[i][2]->enabled = enabled;
233-
reinterpret_cast<OvUI::Widgets::Texts::Text*>(m_materialFields[i][0])->content = m_materialNames[i];
240+
const auto formattedName = std::format("Material [{}]: <{}>", i, m_materialNames[i]);
241+
reinterpret_cast<OvUI::Widgets::Texts::Text*>(m_materialFields[i][0]) ->content = formattedName;
234242
}
235243
}
236244
}

0 commit comments

Comments
 (0)