Improve UI/UX

Author: PicoShot

UnityInspector/src/game/features/inspector/hierarchy_window.cpp

@@ -1,7 +1,6 @@
 #include "pch.h"
 #include "inspector.h"
 
-
 void Inspector::RenderHierarchyNode(HierarchyNode& node, const int depth)
 {
 	if (!Core::helper->SafeIsAlive(node.gameObject)) return;
@@ -22,7 +21,7 @@ void Inspector::RenderHierarchyNode(HierarchyNode& node, const int depth)
 						if (checkChildren(c)) return true;
 					}
 					return false;
-					};
+				};
 				if (checkChildren(child))
 				{
 					matchesSearch = true;
@@ -39,8 +38,11 @@ void Inspector::RenderHierarchyNode(HierarchyNode& node, const int depth)
 	const bool hasChildren = !node.children.empty();
 	const bool isSelected = (FindTabForObject(node.gameObject) >= 0);
 
-	ImGuiTreeNodeFlags flags = ImGuiTreeNodeFlags_OpenOnArrow | ImGuiTreeNodeFlags_SpanAvailWidth;
-	if (!hasChildren) flags |= ImGuiTreeNodeFlags_Leaf;
+	ImGuiTreeNodeFlags flags = ImGuiTreeNodeFlags_OpenOnArrow | 
+	                           ImGuiTreeNodeFlags_SpanAvailWidth |
+	                           ImGuiTreeNodeFlags_FramePadding;
+	
+	if (!hasChildren) flags |= ImGuiTreeNodeFlags_Leaf | ImGuiTreeNodeFlags_NoTreePushOnOpen;
 	if (isSelected) flags |= ImGuiTreeNodeFlags_Selected;
 
 	bool isActive = true;
@@ -50,17 +52,14 @@ void Inspector::RenderHierarchyNode(HierarchyNode& node, const int depth)
 		return;
 	}
 
-	if (!isActive)
-	{
-		ImGui::PushStyleColor(ImGuiCol_Text, ImVec4(0.5f, 0.5f, 0.5f, 1.0f));
-	}
 
-	const bool nodeOpen = ImGui::TreeNodeEx(node.name.c_str(), flags);
+	const char* icon = hasChildren ? " " : "  ";
+	const char* activeIcon = isActive ? "" : " ";
+	
+	std::string label = activeIcon + std::string(icon) + node.name;
+	
+	const bool nodeOpen = ImGui::TreeNodeEx(label.c_str(), flags);
 
-	if (!isActive)
-	{
-		ImGui::PopStyleColor();
-	}
 
 	if (ImGui::IsItemClicked() && !ImGui::IsItemToggledOpen())
 	{
@@ -69,14 +68,19 @@ void Inspector::RenderHierarchyNode(HierarchyNode& node, const int depth)
 
 	if (ImGui::BeginPopupContextItem())
 	{
-		if (ImGui::MenuItem("Inspect"))
+		if (ImGui::MenuItem("Inspect", nullptr, false, true))
 		{
 			OpenObjectInNewTab(node.gameObject);
 		}
+		ImGui::Separator();
+		if (ImGui::MenuItem(isActive ? "Deactivate" : "Activate"))
+		{
+			Core::helper->SafeSetActive(node.gameObject, !isActive);
+		}
 		ImGui::EndPopup();
 	}
 
-	if (nodeOpen)
+	if (hasChildren && nodeOpen)
 	{
 		for (auto& child : node.children)
 		{

UnityInspector/src/game/features/inspector/inspector.cpp

@@ -1,4 +1,4 @@
-#include "pch.h"
+#include "pch.h"
 #include "inspector.h"
 #include "assembly_explorer.h"
 
@@ -44,7 +44,7 @@ void Inspector::Render()
 
 	UR::ThreadAttach();
 
-	ImGui::SetNextWindowSize(ImVec2(350, 500), ImGuiCond_FirstUseEver);
+	ImGui::SetNextWindowSize(ImVec2(380, 600), ImGuiCond_FirstUseEver);
 
 	if (ImGui::Begin("Hierarchy", nullptr))
 	{
@@ -54,19 +54,18 @@ void Inspector::Render()
 		}
 
 		ImGui::SameLine();
-		ImGui::Checkbox("Auto Refresh", &Core::config->inspector.AutoRefresh);
+		ImGui::Checkbox("Auto", &Core::config->inspector.AutoRefresh);
 
 		ImGui::SameLine();
-		ImGui::Text("Objects: %zu", rootNodes.size());
+		ImGui::TextDisabled("| %zu objects", rootNodes.size());
 
 		ImGui::SameLine();
-
 		ImGui::SetNextItemWidth(-1);
 		ImGui::InputTextWithHint("##Search", "Search...", searchBuffer, sizeof(searchBuffer));
 
 		ImGui::Separator();
 
-		if (ImGui::BeginChild("HierarchyTree", ImVec2(0, 0), true))
+		if (ImGui::BeginChild("HierarchyTree", ImVec2(0, 0), false))
 		{
 			if (rootNodes.empty())
 			{
@@ -104,4 +103,4 @@ void Inspector::Render()
 	{
 		assemblyExplorer.reset();
 	}
-}
+}