Move UI Code to the Engine

.gitmodules

@@ -7,9 +7,6 @@
 [submodule "core/lib/spdlog"]
 	path = core/lib/spdlog
 	url = https://github.com/gabime/spdlog
-[submodule "core/lib/imgui"]
-	path = core/lib/imgui
-	url = https://github.com/ocornut/imgui.git
 [submodule "core/lib/fmt"]
 	path = core/lib/fmt
 	url = https://github.com/fmtlib/fmt.git
@@ -25,3 +22,6 @@
 [submodule "docs/css/m.css"]
 	path = docs/css
 	url = https://github.com/mosra/m.css
+[submodule "engine/lib/imgui"]
+	path = engine/lib/imgui
+	url = https://github.com/ocornut/imgui.git

core/CMakeLists.txt

@@ -71,10 +71,6 @@ set(CUBOS_CORE_SOURCE
     "src/cubos/core/al/oal_audio_device.cpp"
     "src/cubos/core/al/oal_audio_device.hpp"
 
-    "src/cubos/core/ui/imgui.cpp"
-    "src/cubos/core/ui/serialization.cpp"
-    "src/cubos/core/ui/ecs.cpp"
-
     "src/cubos/core/ecs/entity_manager.cpp"
     "src/cubos/core/ecs/component_manager.cpp"
     "src/cubos/core/ecs/commands.cpp"
@@ -154,25 +150,14 @@ endif ()
 
 add_subdirectory(lib/json)
 
-add_library(imgui STATIC
-    "lib/imgui/imgui.cpp"
-    "lib/imgui/imgui_draw.cpp"
-    "lib/imgui/imgui_tables.cpp"
-    "lib/imgui/imgui_widgets.cpp"
-    "lib/imgui/imgui_demo.cpp"
-    "lib/imgui/misc/cpp/imgui_stdlib.cpp"
-)
-
-target_include_directories(imgui PUBLIC "lib/imgui" "lib/imgui/misc/cpp/imgui_stdlib.hpp")
-
 add_subdirectory(lib/stduuid)
 target_link_libraries(cubos-core PUBLIC stduuid)
 
 set(THREADS_PREFER_PTHREAD_FLAG ON)
 find_package(Threads REQUIRED)
 
 target_compile_definitions(cubos-core PUBLIC GLM_FORCE_SILENT_WARNINGS) # Needed for compilation to succeed on MSVC
-target_link_libraries(cubos-core PUBLIC glm::glm spdlog imgui fmt::fmt json ${CMAKE_DL_LIBS})
+target_link_libraries(cubos-core PUBLIC glm::glm spdlog fmt::fmt json ${CMAKE_DL_LIBS})
 target_link_libraries(cubos-core PRIVATE glad Threads::Threads)
 
 # Add core tests

core/samples/CMakeLists.txt

@@ -37,4 +37,3 @@ make_sample(DIR "ecs/general")
 make_sample(DIR "gl/compute")
 make_sample(DIR "gl/debug_renderer")
 make_sample(DIR "gl/quad")
-make_sample(DIR "ui")

engine/CMakeLists.txt

@@ -19,6 +19,8 @@ set(CUBOS_ENGINE_SOURCE
     "src/cubos/engine/window/plugin.cpp"
 
     "src/cubos/engine/imgui/plugin.cpp"
+    "src/cubos/engine/imgui/imgui.cpp"
+    "src/cubos/engine/imgui/serialization.cpp"
 
     "src/cubos/engine/tools/asset_explorer/plugin.cpp"
     "src/cubos/engine/tools/settings_inspector/plugin.cpp"
@@ -67,6 +69,18 @@ target_include_directories(cubos-engine PUBLIC "include" PRIVATE "src")
 target_link_libraries(cubos-engine PUBLIC cubos-core)
 cubos_common_target_options(cubos-engine)
 
+add_library(imgui STATIC
+    "lib/imgui/imgui.cpp"
+    "lib/imgui/imgui_draw.cpp"
+    "lib/imgui/imgui_tables.cpp"
+    "lib/imgui/imgui_widgets.cpp"
+    "lib/imgui/imgui_demo.cpp"
+    "lib/imgui/misc/cpp/imgui_stdlib.cpp"
+)
+
+target_include_directories(imgui PUBLIC "lib/imgui" "lib/imgui/misc/cpp/imgui_stdlib.hpp")
+target_link_libraries(cubos-engine PUBLIC imgui)
+
 if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
     target_compile_options(cubos-engine PUBLIC -Wno-attributes)
 endif()

core/include/cubos/core/ui/serialization.hpp → engine/include/cubos/engine/imgui/serialization.hpp

@@ -1,12 +1,12 @@
 /// @file
 /// @brief Functions for showing and editing serializable objects in the UI.
-/// @ingroup core-ui
+/// @ingroup imgui-plugin
 
 #pragma once
 
 #include <cubos/core/data/package.hpp>
 
-namespace cubos::core::ui
+namespace cubos::engine
 {
     /// @brief Shows a packaged object's properties in the UI. Should be called inside a
     /// `ImGui::BeginTable(2)` and `ImGui::EndTable()` block.
@@ -16,8 +16,8 @@ namespace cubos::core::ui
     ///
     /// @param pkg Packaged object to show.
     /// @param name Name of the object.
-    /// @ingroup core-ui
-    void showPackage(const data::Package& pkg, const std::string& name);
+    /// @ingroup imgui-plugin
+    void imguiShowPackage(const core::data::Package& pkg, const std::string& name);
 
     /// @brief Shows a packaged object's properties in the UI, allowing the user to edit the
     /// object. Should be called inside a `ImGui::BeginTable(3)` and `ImGui::EndTable()` block.
@@ -29,8 +29,8 @@ namespace cubos::core::ui
     /// @param pkg Packaged object to edit.
     /// @param name Name of the object.
     /// @return True if the object was modified, false otherwise.
-    /// @ingroup core-ui
-    bool editPackage(data::Package& pkg, const std::string& name);
+    /// @ingroup imgui-plugin
+    bool imguiEditPackage(core::data::Package& pkg, const std::string& name);
 
     /// @brief Shows a serializable object's properties in the UI. Should be called inside a
     /// `ImGui::BeginTable(2)` and `ImGui::EndTable()` block.
@@ -40,12 +40,12 @@ namespace cubos::core::ui
     /// @tparam T Type of the serializable object.
     /// @param object Object to show.
     /// @param name Name of the object.
-    /// @ingroup core-ui
+    /// @ingroup imgui-plugin
     template <typename T>
-    inline void show(const T& object, const std::string& name)
+    inline void imguiShow(const T& object, const std::string& name)
     {
-        auto pkg = data::Package::from(object);
-        showPackage(pkg, name);
+        auto pkg = core::data::Package::from(object);
+        imguiShowPackage(pkg, name);
     }
 
     /// @brief Shows a serializable object's properties in the UI. Should be called inside a
@@ -58,17 +58,17 @@ namespace cubos::core::ui
     /// @param object Object to edit.
     /// @param name Name of the object.
     /// @return True if the object was edited, false otherwise.
-    /// @ingroup core-ui
+    /// @ingroup imgui-plugin
     template <typename T>
-    inline bool edit(T& object, const std::string& name)
+    inline bool imguiEdit(T& object, const std::string& name)
     {
-        auto pkg = data::Package::from(object);
-        if (editPackage(pkg, name))
+        auto pkg = core::data::Package::from(object);
+        if (imguiEditPackage(pkg, name))
         {
             pkg.into(object);
             return true;
         }
 
         return false;
     }
-} // namespace cubos::core::ui
+} // namespace cubos::engine