From f6c5ace0999fb9d5309dd8b0e5b3c0371841367d Mon Sep 17 00:00:00 2001
From: scawful <justin.scofield@outlook.com>
Date: Tue, 14 Oct 2025 09:43:54 -0400
Subject: [PATCH] refactor(editor): streamline menu item separator handling in
 EditorManager

- Adjusted the order of method calls in BuildModernMenu to improve readability and maintainability.
- Moved the separator method call to follow the item addition for better logical flow.

Benefits:
- Enhances code clarity and organization within the menu building logic of the EditorManager.
---
 src/app/editor/editor_manager.cc | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/app/editor/editor_manager.cc b/src/app/editor/editor_manager.cc
index af458756..01be203a 100644
--- a/src/app/editor/editor_manager.cc
+++ b/src/app/editor/editor_manager.cc
@@ -1317,9 +1317,8 @@ void EditorManager::BuildModernMenu() {
           "Proposal Drawer", ICON_MD_PREVIEW,
           [this]() { proposal_drawer_.Toggle(); }, "Ctrl+P")
 #endif
-
-  menu_builder_.Separator()
-      .Item(
+      .Separator();
+  menu_builder_.Item(
           "Card Browser", ICON_MD_DASHBOARD,
           [this]() { show_card_browser_ = true; }, "Ctrl+Shift+B")
       .Separator()