From 71916b5e28dacdaa5053b98fbba64e192a2722ff Mon Sep 17 00:00:00 2001
From: zadam <zadam.apps@gmail.com>
Date: Sun, 14 Apr 2019 13:51:21 +0200
Subject: [PATCH] empty icon for some menu actions do de-emphasize less
 important items

---
 .../javascripts/services/tree_context_menu.js | 10 +++----
 src/public/stylesheets/style.css              |  5 ++++
 src/views/desktop.ejs                         | 26 +++++++++----------
 3 files changed, 23 insertions(+), 18 deletions(-)

diff --git a/src/public/javascripts/services/tree_context_menu.js b/src/public/javascripts/services/tree_context_menu.js
index 8a3aeecda..3c1fd601a 100644
--- a/src/public/javascripts/services/tree_context_menu.js
+++ b/src/public/javascripts/services/tree_context_menu.js
@@ -112,9 +112,9 @@ async function getTopLevelItems(event) {
         { title: "Delete <kbd>Delete</kbd>", cmd: "delete", uiIcon: "trash",
             enabled: isNotRoot && parentNote.type !== 'search' },
         { title: "----" },
-        isHoisted ? null : { title: "Hoist note <kbd>Ctrl-H</kbd>", cmd: "hoist", uiIcon: "arrow-up" },
+        isHoisted ? null : { title: "Hoist note <kbd>Ctrl-H</kbd>", cmd: "hoist", uiIcon: "empty" },
         !isHoisted || !isNotRoot ? null : { title: "Unhoist note <kbd>Ctrl-H</kbd>", cmd: "unhoist", uiIcon: "arrow-up" },
-        { title: "Edit branch prefix <kbd>F2</kbd>", cmd: "editBranchPrefix", uiIcon: "pencil",
+        { title: "Edit branch prefix <kbd>F2</kbd>", cmd: "editBranchPrefix", uiIcon: "empty",
             enabled: isNotRoot && parentNote.type !== 'search'},
         { title: "----" },
         { title: "Protect subtree", cmd: "protectSubtree", uiIcon: "shield-check" },
@@ -129,14 +129,14 @@ async function getTopLevelItems(event) {
         { title: "Paste after", cmd: "pasteAfter", uiIcon: "clipboard",
             enabled: clipboardIds.length > 0 && isNotRoot && parentNote.type !== 'search' },
         { title: "----" },
-        { title: "Export", cmd: "export", uiIcon: "arrow-up-right",
+        { title: "Export", cmd: "export", uiIcon: "empty",
             enabled: note.type !== 'search' },
-        { title: "Import into note", cmd: "importIntoNote", uiIcon: "arrow-down-left",
+        { title: "Import into note", cmd: "importIntoNote", uiIcon: "empty",
             enabled: note.type !== 'search' },
         { title: "----" },
         { title: "Collapse subtree <kbd>Alt+-</kbd>", cmd: "collapseSubtree", uiIcon: "align-justify" },
         { title: "Force note sync", cmd: "forceNoteSync", uiIcon: "refresh" },
-        { title: "Sort alphabetically <kbd>Alt+S</kbd>", cmd: "sortAlphabetically", uiIcon: "arrows-v" }
+        { title: "Sort alphabetically <kbd>Alt+S</kbd>", cmd: "sortAlphabetically", uiIcon: "empty" }
     ].filter(row => row !== null);
 }
 
diff --git a/src/public/stylesheets/style.css b/src/public/stylesheets/style.css
index 453338d97..d6829ed31 100644
--- a/src/public/stylesheets/style.css
+++ b/src/public/stylesheets/style.css
@@ -928,4 +928,9 @@ a.external:after, a[href^="http://"]:after, a[href^="https://"]:after {
 .card {
     background-color: inherit !important;
     border-color: var(--main-border-color) !important;
+}
+
+.jam-empty {
+    width: 1em;
+    display: inline-block;
 }
\ No newline at end of file
diff --git a/src/views/desktop.ejs b/src/views/desktop.ejs
index 0d83ed2e6..636443894 100644
--- a/src/views/desktop.ejs
+++ b/src/views/desktop.ejs
@@ -61,32 +61,32 @@
                         Sync (<span id="outstanding-syncs-count">0</span>)
                     </a>
 
-                    <a class="dropdown-item" id="show-help-button">
-                        <span class="jam jam-help"></span>
-                        Show Help
-                        <kbd>F1</kbd>
-                    </a>
-
                     <a class="dropdown-item" id="open-dev-tools-button">
                         <span class="jam jam-terminal"></span>
                         Open Dev Tools
                         <kbd>CTRL+SHIFT+I</kbd>
                     </a>
 
-                    <a class="dropdown-item" id="reload-frontend-button" title="Reload can help with some visual glitches without restarting the whole app.">
-                        <span class="jam jam-refresh"></span>
-                        Reload frontend
-                        <kbd>CTRL-R</kbd>
-                    </a>
-
                     <a class="dropdown-item" id="open-sql-console-button">
                         <span class="jam jam-database"></span>
                         Open SQL Console
                         <kbd>ALT+O</kbd>
                     </a>
 
+                    <a class="dropdown-item" id="reload-frontend-button" title="Reload can help with some visual glitches without restarting the whole app.">
+                        <span class="jam jam-empty"></span>
+                        Reload frontend
+                        <kbd>CTRL-R</kbd>
+                    </a>
+
+                    <a class="dropdown-item" id="show-help-button">
+                        <span class="jam jam-help"></span>
+                        Show Help
+                        <kbd>F1</kbd>
+                    </a>
+
                     <a class="dropdown-item" id="show-about-dialog-button">
-                        <span class="jam jam-info"></span>
+                        <span class="jam jam-empty"></span>
                         About Trilium Notes
                     </a>