From 2dc864bb5cfaabce41eb9c9a1440e8d385be9a0f Mon Sep 17 00:00:00 2001
From: Joshua Goins <josh@redstrate.com>
Date: Sat, 3 Feb 2024 10:34:05 -0500
Subject: [PATCH] armoury: Separate and improve menu structure, add icons

---
 armoury/src/mainwindow.cpp | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/armoury/src/mainwindow.cpp b/armoury/src/mainwindow.cpp
index e1644eb..6b2586d 100644
--- a/armoury/src/mainwindow.cpp
+++ b/armoury/src/mainwindow.cpp
@@ -63,8 +63,18 @@ void MainWindow::setupAdditionalMenus(QMenuBar *menuBar)
 {
     auto toolsMenu = menuBar->addMenu(QStringLiteral("Tools"));
 
-    auto fmvMenu = toolsMenu->addAction(QStringLiteral("Full Model viewer"));
+    auto cmpEditorMenu = toolsMenu->addAction(QStringLiteral("CMP Editor"));
+    cmpEditorMenu->setIcon(QIcon::fromTheme(QStringLiteral("document-edit")));
+    connect(cmpEditorMenu, &QAction::triggered, [this] {
+        auto cmpEditor = new CmpEditor(&data);
+        cmpEditor->show();
+    });
+
+    auto windowMenu = menuBar->addMenu(QStringLiteral("Window"));
+
+    auto fmvMenu = windowMenu->addAction(QStringLiteral("Full Model viewer"));
     fmvMenu->setCheckable(true);
+    fmvMenu->setIcon(QIcon::fromTheme(QStringLiteral("user-symbolic")));
     connect(fmvMenu, &QAction::toggled, [this](bool toggled) {
         if (toggled) {
             fullModelViewer->show();
@@ -76,15 +86,11 @@ void MainWindow::setupAdditionalMenus(QMenuBar *menuBar)
         fmvMenu->setChecked(fullModelViewer->isVisible());
     });
 
-    auto cmpEditorMenu = toolsMenu->addAction(QStringLiteral("CMP Editor"));
-    cmpEditorMenu->setIcon(QIcon::fromTheme(QStringLiteral("document-edit")));
-    connect(cmpEditorMenu, &QAction::triggered, [this] {
-        auto cmpEditor = new CmpEditor(&data);
-        cmpEditor->show();
-    });
+    auto settingsMenu = menuBar->addMenu(QStringLiteral("Settings"));
 
-    auto settingsMenu = toolsMenu->addAction(QStringLiteral("Settings..."));
-    connect(settingsMenu, &QAction::triggered, [this] {
+    auto settingsAction = settingsMenu->addAction(QStringLiteral("Configure Armoury..."));
+    settingsAction->setIcon(QIcon::fromTheme(QStringLiteral("configure-symbolic")));
+    connect(settingsAction, &QAction::triggered, [this] {
         auto settingsWindow = new SettingsWindow();
         settingsWindow->show();
     });