Merge pull request #1856 from alicevision/dev/reorgFileMenu

[ui] Reorganize the "File" menu
alicevision · Dec 22, 2022 · bd792bf · bd792bf
2 parents 9ac21d5 + eb4ec83
commit bd792bf
Showing 1 changed file with 58 additions and 40 deletions.
diff --git a/meshroom/ui/qml/main.qml b/meshroom/ui/qml/main.qml
@@ -68,6 +68,13 @@ ApplicationWindow {
         settings_General.windowHeight = _window.height
     }
 
+    Shortcut {
+        // Ensures the Ctrl+N shortcut is always valid and creates a default pipeline
+        sequence: "Ctrl+N"
+        context: Qt.ApplicationShortcut
+        onActivated: ensureSaved(function() { _reconstruction.new() })
+    }
+
     MessageDialog {
         id: unsavedDialog
 
@@ -483,11 +490,6 @@ ApplicationWindow {
         palette.window: Qt.darker(activePalette.window, 1.15)
         Menu {
             title: "File"
-            Action {
-                text: "New"
-                shortcut: "Ctrl+N"
-                onTriggered: ensureSaved(function() { _reconstruction.new() })
-            }
             Menu {
                 id: newPipelineMenu
                 title: "New Pipeline"
@@ -572,15 +574,33 @@ ApplicationWindow {
                     }
                 }
             }
+            MenuSeparator { }
+            Action {
+                id: saveAction
+                text: "Save"
+                shortcut: "Ctrl+S"
+                enabled: _reconstruction ? (_reconstruction.graph && !_reconstruction.graph.filepath) || !_reconstruction.undoStack.clean : false
+                onTriggered: {
+                    if(_reconstruction.graph.filepath) {
+                        _reconstruction.save();
+                    }
+                    else
+                    {
+                        initFileDialogFolder(saveFileDialog);
+                        saveFileDialog.open();
+                    }
+                }
+            }
             Action {
-                id: importProjectAction
-                text: "Import Project"
-                shortcut: "Ctrl+Shift+I"
+                id: saveAsAction
+                text: "Save As..."
+                shortcut: "Ctrl+Shift+S"
                 onTriggered: {
-                    initFileDialogFolder(importProjectDialog);
-                    importProjectDialog.open();
+                    initFileDialogFolder(saveFileDialog);
+                    saveFileDialog.open();
                 }
             }
+            MenuSeparator { }
             Action {
                 id: importImagesAction
                 text: "Import Images"
@@ -613,39 +633,37 @@ ApplicationWindow {
                     }
                 }
             }
-
-            Action {
-                id: saveAction
-                text: "Save"
-                shortcut: "Ctrl+S"
-                enabled: _reconstruction ? (_reconstruction.graph && !_reconstruction.graph.filepath) || !_reconstruction.undoStack.clean : false
-                onTriggered: {
-                    if(_reconstruction.graph.filepath) {
-                        _reconstruction.save()
+            MenuSeparator { }
+            Menu {
+                id: advancedMenu
+                title: "Advanced"
+
+                Action {
+                    id: saveAsTemplateAction
+                    text: "Save As Template..."
+                    shortcut: Shortcut {
+                        sequence: "Ctrl+Shift+T"
+                        context: Qt.ApplicationShortcut
+                        onActivated: saveAsTemplateAction.triggered()
                     }
-                    else
-                    {
-                        initFileDialogFolder(saveFileDialog);
-                        saveFileDialog.open();
+                    onTriggered: {
+                            initFileDialogFolder(saveTemplateDialog);
+                            saveTemplateDialog.open();
                     }
                 }
-            }
-            Action {
-                id: saveAsAction
-                text: "Save As..."
-                shortcut: "Ctrl+Shift+S"
-                onTriggered: {
-                    initFileDialogFolder(saveFileDialog);
-                    saveFileDialog.open();
-                }
-            }
-            Action {
-                id: saveAsTemplateAction
-                text: "Save As Template..."
-                shortcut: "Ctrl+Shift+T"
-                onTriggered: {
-                    initFileDialogFolder(saveTemplateDialog);
-                    saveTemplateDialog.open();
+
+                Action {
+                    id: importProjectAction
+                    text: "Import Project"
+                    shortcut: Shortcut {
+                        sequence: "Ctrl+Shift+I"
+                        context: Qt.ApplicationShortcut
+                        onActivated: importProjectAction.triggered()
+                    }
+                    onTriggered: {
+                        initFileDialogFolder(importProjectDialog);
+                        importProjectDialog.open();
+                    }
                 }
             }
             MenuSeparator { }