Simplify inspector code for tracking

CMakeLists.txt

@@ -58,6 +58,7 @@ add_executable(Prune
         src/prune/tooling/inspector.cpp
         src/prune/tooling/command_history.cpp
         src/prune/tooling/sprite_picker.cpp
+        src/prune/tooling/editor/tracked_property_table.cpp
         src/prune/tooling/controls.cpp
         src/prune/tooling/options.cpp
         src/prune/tooling/stats.cpp

src/prune/editor/editor_command.cpp

@@ -110,50 +110,60 @@ namespace prune {
 
     EditorCommand make_object_command(
         EditorCommandType type,
-        std::string label,
+        std::string_view label,
         const GameObject& before,
-        const GameObject& after
+        const GameObject& after,
+        std::string_view detail
     ) {
         EditorCommand command{};
         command.type = type;
-        command.label = std::move(label);
+        command.label = std::string(label);
+        command.detail = std::string(detail);
         command.object_id = after.identity.id;
         command.before_object = before;
         command.after_object = after;
 
         return command;
     }
 
-    EditorCommand make_create_object_command(const GameObject& created)
-    {
+    EditorCommand make_create_object_command(
+        const GameObject& created,
+        std::string_view detail
+    ) {
         EditorCommand command{};
         command.type = EditorCommandType::CreateObject;
         command.label = editor_command_type_label(EditorCommandType::CreateObject);
+        command.detail = std::string(detail);
         command.object_id = created.identity.id;
         command.after_object = created;
 
         return command;
     }
 
-    EditorCommand make_delete_object_command(const GameObject& deleted)
-    {
+    EditorCommand make_delete_object_command(
+        const GameObject& deleted,
+        std::string_view detail
+    ) {
         EditorCommand command{};
         command.type = EditorCommandType::DeleteObject;
         command.label = editor_command_type_label(EditorCommandType::DeleteObject);
+        command.detail = std::string(detail);
         command.object_id = deleted.identity.id;
         command.before_object = deleted;
 
         return command;
     }
 
     EditorCommand make_viewport_command(
-        std::string label,
+        std::string_view label,
         const Camera& before,
-        const Camera& after
+        const Camera& after,
+        std::string_view detail
     ) {
         EditorCommand command{};
         command.type = EditorCommandType::MoveViewport;
-        command.label = std::move(label);
+        command.label = std::string(label);
+        command.detail = std::string(detail);
         command.before_camera = before;
         command.after_camera = after;
 

src/prune/editor/editor_command.hpp

@@ -3,6 +3,7 @@
 #include <cstddef>
 #include <optional>
 #include <string>
+#include <string_view>
 #include <vector>
 
 #include "prune/scene/game_object.hpp"
@@ -27,6 +28,7 @@ namespace prune {
     struct EditorCommand {
         EditorCommandType type = EditorCommandType::MoveObject;
         std::string label;
+        std::string detail;
         GameObjectId object_id = k_invalid_game_object_id;
 
         std::optional<GameObject> before_object;
@@ -62,18 +64,27 @@ namespace prune {
 
     [[nodiscard]] EditorCommand make_object_command(
         EditorCommandType type,
-        std::string label,
+        std::string_view label,
         const GameObject& before,
-        const GameObject& after
+        const GameObject& after,
+        std::string_view detail = {}
     );
 
-    [[nodiscard]] EditorCommand make_create_object_command(const GameObject& created);
-    [[nodiscard]] EditorCommand make_delete_object_command(const GameObject& deleted);
+    [[nodiscard]] EditorCommand make_create_object_command(
+        const GameObject& created,
+        std::string_view detail = {}
+    );
+
+    [[nodiscard]] EditorCommand make_delete_object_command(
+        const GameObject& deleted,
+        std::string_view detail = {}
+    );
 
     [[nodiscard]] EditorCommand make_viewport_command(
-        std::string label,
+        std::string_view label,
         const Camera& before,
-        const Camera& after
+        const Camera& after,
+        std::string_view detail = {}
     );
 
 }

src/prune/scene/scene_interaction.cpp

@@ -54,7 +54,8 @@ namespace prune {
                     scene.record_editor_command(make_viewport_command(
                         editor_command_type_label(EditorCommandType::MoveViewport),
                         m_viewport_pan_start,
-                        after
+                        after,
+                        "Mouse pan"
                     ));
                 }
 
@@ -110,7 +111,8 @@ namespace prune {
                             EditorCommandType::MoveObject,
                             editor_command_type_label(EditorCommandType::MoveObject),
                             before,
-                            *object
+                            *object,
+                            "Mouse drag"
                         ));
                     }
                 }
@@ -233,7 +235,8 @@ namespace prune {
                 EditorCommandType::MoveObject,
                 editor_command_type_label(EditorCommandType::MoveObject),
                 before,
-                *selected
+                *selected,
+                "Keyboard nudge"
             ));
         }
     }

src/prune/scene/world_scene.cpp

@@ -97,7 +97,7 @@ namespace prune {
             if (ImGui::Button(action.label.data())) {
                 const GameObjectId created_id = create_scene_object(action.id);
                 if (const GameObject* created = m_state.objects.get_by_id(created_id)) {
-                    record_editor_command(make_create_object_command(*created));
+                    record_editor_command(make_create_object_command(*created, action.label));
                 }
             }
             ImGui::PopID();

src/prune/tooling/command_history.cpp

@@ -48,14 +48,26 @@ namespace prune {
                     ImGui::BeginDisabled();
                 }
 
-                ImGui::Text(
-                    "%s%zu. %s",
-                    applied ? "" : "redo: ",
-                    command_index + 1,
-                    command.label.empty()
-                        ? editor_command_type_label(command.type)
-                        : command.label.c_str()
-                );
+                const char* label = command.label.empty()
+                    ? editor_command_type_label(command.type)
+                    : command.label.c_str();
+
+                if (command.detail.empty()) {
+                    ImGui::Text(
+                        "%s%zu. %s",
+                        applied ? "" : "redo: ",
+                        command_index + 1,
+                        label
+                    );
+                } else {
+                    ImGui::Text(
+                        "%s%zu. %s: %s",
+                        applied ? "" : "redo: ",
+                        command_index + 1,
+                        label,
+                        command.detail.c_str()
+                    );
+                }
 
                 if (command.object_id != k_invalid_game_object_id) {
                     ImGui::SameLine();