feat: docking split init

Author: jcardonne

.gitignore

@@ -173,3 +173,9 @@ _CPack_Packages/
 
 myinstall/
 *.gcov
+
+.nexo/search_history.json
+
+.task/checksum/configure
+
+Taskfile.yml

editor/src/DocumentWindows/EditorScene/EditorScene.hpp

@@ -126,10 +126,6 @@ namespace nexo::editor
         bool m_shouldFocusGameWindow = false;
         std::string m_gameWindowToFocus;
 
-        // Deferred dock split operation
-        bool m_shouldSplitDock = false;
-        std::string m_gameWindowNameToSplit;
-
         // Selected button gradient - lighter blue gradient
         const std::vector<ImNexo::GradientStop> m_selectedGradient = {
             {0.0f, IM_COL32(70, 70, 120, 230)},

editor/src/DocumentWindows/EditorScene/Show.cpp

@@ -163,34 +163,6 @@ namespace nexo::editor
 
     void EditorScene::show()
     {
-        // Handle deferred dock split before rendering
-        if (m_shouldSplitDock && !m_gameWindowNameToSplit.empty())
-        {
-            const std::string currentWindowName = m_windowName;
-            const ImGuiWindow *currentImGuiWindow = ImGui::FindWindowByName((currentWindowName + NEXO_WND_USTRID_DEFAULT_SCENE + std::to_string(m_sceneId)).c_str());
-
-            if (currentImGuiWindow && currentImGuiWindow->DockId)
-            {
-                const ImGuiID editorDockId = currentImGuiWindow->DockId;
-                ImGuiID rightNode, leftNode;
-
-                if (ImGui::DockBuilderSplitNode(editorDockId, ImGuiDir_Right, 0.5f, &rightNode, &leftNode))
-                {
-                    // Dock the windows
-                    ImGui::DockBuilderDockWindow((currentWindowName + NEXO_WND_USTRID_DEFAULT_SCENE + std::to_string(m_sceneId)).c_str(), leftNode);
-                    ImGui::DockBuilderDockWindow(m_gameWindowNameToSplit.c_str(), rightNode);
-                    ImGui::DockBuilderFinish(editorDockId);
-
-                    // Update registry
-                    m_windowRegistry.setDockId(currentWindowName + NEXO_WND_USTRID_DEFAULT_SCENE + std::to_string(m_sceneId), leftNode);
-                    m_windowRegistry.setDockId(m_gameWindowNameToSplit, rightNode);
-                }
-            }
-
-            m_shouldSplitDock = false;
-            m_gameWindowNameToSplit.clear();
-        }
-
         ImGui::PushStyleVar(ImGuiStyleVar_WindowPadding, ImVec2(0, 0));
         ImGui::SetNextWindowSizeConstraints(ImVec2(480, 270), ImVec2(1920, 1080));
         auto& selector = Selector::get();

editor/src/DocumentWindows/EditorScene/Toolbar.cpp

@@ -59,11 +59,16 @@ namespace nexo::editor {
                     newGameWindow->setup();
                     newGameWindow->setOpened(true);
 
-                    // Schedule dock split for next frame
-                    m_shouldSplitDock = true;
-                    m_gameWindowNameToSplit = gameWindowName;
-
-                    // Also schedule focus for after the split
+                    // Split the dock and attach the new game window
+                    const std::string editorWindowFullName = std::format("{}{}{}", m_windowName, NEXO_WND_USTRID_DEFAULT_SCENE, m_sceneId);
+                    m_windowRegistry.splitDockAndAttachWindow(
+                        editorWindowFullName,
+                        gameWindowName,
+                        ImGuiDir_Right,
+                        0.5f
+                    );
+
+                    // Schedule focus for the new window
                     m_shouldFocusGameWindow = true;
                     m_gameWindowToFocus = gameWindowName;
                 }

editor/src/WindowRegistry.cpp

@@ -17,6 +17,8 @@
 ///////////////////////////////////////////////////////////////////////////////
 
 #include "WindowRegistry.hpp"
+#include "Logger.hpp"
+#include <imgui_internal.h>
 
 namespace nexo::editor {
 
@@ -93,4 +95,38 @@ namespace nexo::editor {
             }
         }
 	}
+
+	bool WindowRegistry::splitDockAndAttachWindow(
+		const std::string& sourceWindowName,
+		const std::string& newWindowName,
+		const ImGuiDir splitDirection,
+		const float splitRatio)
+	{
+		const ImGuiWindow* sourceImGuiWindow = ImGui::FindWindowByName(sourceWindowName.c_str());
+
+		if (!sourceImGuiWindow || !sourceImGuiWindow->DockId)
+		{
+			LOG(NEXO_WARN, "Cannot split dock: source window '{}' not found or not docked", sourceWindowName);
+			return false;
+		}
+
+		const ImGuiID sourceDockId = sourceImGuiWindow->DockId;
+		ImGuiID newNode, remainingNode;
+
+		if (!ImGui::DockBuilderSplitNode(sourceDockId, splitDirection, splitRatio, &newNode, &remainingNode))
+		{
+			LOG(NEXO_WARN, "Failed to split dock node for window '{}'", sourceWindowName);
+			return false;
+		}
+
+		ImGui::DockBuilderDockWindow(sourceWindowName.c_str(), remainingNode);
+		ImGui::DockBuilderDockWindow(newWindowName.c_str(), newNode);
+		ImGui::DockBuilderFinish(sourceDockId);
+
+		m_dockingRegistry.setDockId(sourceWindowName, remainingNode);
+		m_dockingRegistry.setDockId(newWindowName, newNode);
+
+		LOG(NEXO_DEBUG, "Successfully split dock for '{}' and attached '{}'", sourceWindowName, newWindowName);
+		return true;
+	}
 }

editor/src/WindowRegistry.hpp

@@ -268,6 +268,26 @@ namespace nexo::editor {
              */
 			void resetDockId(const std::string &name);
 
+			/**
+			 * @brief Splits a dock node and attaches a new window to the split.
+			 *
+			 * Finds the source window's dock node, splits it in the specified direction,
+			 * and docks both windows to the resulting nodes. Updates the docking registry
+			 * with the new dock IDs.
+			 *
+			 * @param sourceWindowName Name of the window whose dock will be split
+			 * @param newWindowName Name of the window to dock in the new split
+			 * @param splitDirection Direction to split (ImGuiDir_Left/Right/Up/Down)
+			 * @param splitRatio Ratio of the split (0.0-1.0, typically 0.5 for 50/50)
+			 * @return true if split was successful, false if source window not found
+			 */
+			bool splitDockAndAttachWindow(
+				const std::string& sourceWindowName,
+				const std::string& newWindowName,
+				ImGuiDir splitDirection,
+				float splitRatio
+			);
+
 			/**
 			 * @brief Initializes all managed windows.
 			 *