fix: breadcrumb ID conflict and empty labels on dictionary navigation (#23)

* fix missing preposition

* fix: breadcrumb ID conflict and empty labels on dictionary navigation

- Add PushID/PopID around breadcrumb buttons to avoid conflicting IDs when stack has duplicate names

- Protect against empty runtime field/property/method names by checking first character

- Fall back to (unnamed) label in breadcrumb when name string is empty

Author: juzt3

UnityInspector/src/features/inspector/inspector_window.cpp

@@ -2566,16 +2566,21 @@ void Inspector::RenderTabContent(InspectedObjectTab& tab)
 	ImGui::PushStyleVar(ImGuiStyleVar_ItemSpacing, ImVec2(4, 4));
 	for (size_t i = 0; i < tab.navigationStack.size(); ++i)
 	{
+		ImGui::PushID(static_cast<int>(i));
 		if (i > 0)
 		{
 			ImGui::SameLine();
 			ImGui::TextDisabled(">");
 			ImGui::SameLine();
 		}
-		if (ImGui::Button(tab.navigationStack[i].name.c_str()))
+		const char* label = tab.navigationStack[i].name.c_str();
+		if (label[0] == '\0')
+			label = "(unnamed)";
+		if (ImGui::Button(label))
 		{
 			tab.navigationStack.resize(i + 1);
 		}
+		ImGui::PopID();
 	}
 	ImGui::PopStyleVar();
 	ImGui::Separator();

UnityInspector/src/features/inspector/utils.cpp

@@ -43,7 +43,7 @@ std::vector<ComponentFieldInfo> Inspector::GetObjectFields(void* obj, void* klas
 			info.classHandle = currentClass;
 
 			const char* fieldName = UR::Invoke<const char*, void*>(API("field_get_name"), field);
-			info.name = fieldName ? fieldName : "(unknown)";
+			info.name = fieldName && fieldName[0] ? fieldName : "(unnamed)";
 
 			info.offset = UR::Invoke<int, void*>(API("field_get_offset"), field);
 
@@ -117,7 +117,7 @@ std::vector<ComponentPropertyInfo> Inspector::GetObjectProperties(void* obj, voi
 			ComponentPropertyInfo info;
 
 			const char* propName = UR::Invoke<const char*, void*>(API("property_get_name"), prop);
-			info.name = propName ? propName : "(unknown)";
+			info.name = propName && propName[0] ? propName : "(unnamed)";
 
 			info.getterHandle = UR::Invoke<void*, void*>(API("property_get_get_method"), prop);
 			info.setterHandle = UR::Invoke<void*, void*>(API("property_get_set_method"), prop);
@@ -181,7 +181,7 @@ std::vector<ComponentMethodInfo> Inspector::GetObjectMethods(void* obj, void* kl
 			info.methodHandle = method;
 
 			const char* methodName = UR::Invoke<const char*, void*>(API("method_get_name"), method);
-			info.name = methodName ? methodName : "(unknown)";
+			info.name = methodName && methodName[0] ? methodName : "(unnamed)";
 
 			int fFlags = 0;
 			info.flags = UR::Invoke<int, void*, int*>(API("method_get_flags"), method, &fFlags);