Consolidate all main nav items in a single partial

3 files changed, 37 insertions, 25 deletions

diff --git a/app/views/layouts/_nav_controls.html.erb b/app/views/layouts/_nav_controls.html.erb
new file mode 100644
index 00000000..d6053dab
--- /dev/null
+++ b/app/views/layouts/_nav_controls.html.erb
@@ -0,0 +1,36 @@
+<% if current_user %>
+  <%= link_to icon("home", library: "tabler", "aria-hidden": true), admin_path, "aria-label": t(".dashboard"), title: t(".dashboard") %>
+  <a href="#" onclick="admin_search.display_toggle(); return false;" aria-label="<%= t(".search") %>" title="<%= t(".search") %>"><%= icon("search", library: "tabler", "aria-hidden": true) %></a>
+<% end %>
+
+<span class="nav_group">
+  <%= link_to url_for(locale: (:en unless I18n.locale == :en)),
+        class: "nav_toggle nav_locale",
+        "aria-label" => t(".switch_locale"), title: t(".switch_locale") do %>
+    <%= I18n.locale == :en ? "DE" : "EN" %>
+  <% end %>
+  <%= check_box_tag "scheme_override", 1, false, class: "nav_toggle_input" %>
+  <%= label_tag "scheme_override", class: "nav_toggle",
+        "aria-label" => t(".scheme_override"), title: t(".scheme_override") do %>
+    <%= icon("contrast", library: "tabler", "aria-hidden": true) %>
+  <% end %>
+</span>
+
+<%= javascript_tag nonce: true do %>
+  (function () {
+    var box = document.getElementById("scheme_override");
+    try {
+      if (localStorage.getItem("override-prefers-color-scheme")) box.checked = true;
+      box.addEventListener("change", function () {
+        if (box.checked) localStorage.setItem("override-prefers-color-scheme", 1);
+        else localStorage.removeItem("override-prefers-color-scheme");
+      });
+    } catch (e) { /* storage unavailable: the toggle still works per page */ }
+  })();
+<% end %>
+
+<% if current_user %>
+  <%= button_to logout_path, method: :delete, aria: { label: t(".log_out") }, title: t(".log_out") do %>
+    <%= icon("logout", library: "tabler", "aria-hidden": true) %>
+  <% end %>
+<% end %>
diff --git a/app/views/layouts/_nav_switchers.html.erb b/app/views/layouts/_nav_switchers.html.erb
deleted file mode 100644
index 9c3b222e..00000000
--- a/app/views/layouts/_nav_switchers.html.erb
+++ /dev/null
@@ -1,23 +0,0 @@
-<%= link_to url_for(locale: (:en unless I18n.locale == :en)),
-      class: "nav_toggle",
-      "aria-label" => t(".switch_locale"), title: t(".switch_locale") do %>
-  <%= I18n.locale == :en ? "DE" : "EN" %>
-<% end %>
-<%= check_box_tag "scheme_override", 1, false, class: "nav_toggle_input" %>
-<%= label_tag "scheme_override", class: "nav_toggle",
-      "aria-label" => t(".scheme_override"), title: t(".scheme_override") do %>
-  <%= icon("contrast", library: "tabler", "aria-hidden": true) %>
-<% end %>
-
-<%= javascript_tag nonce: true do %>
-  (function () {
-    var box = document.getElementById("scheme_override");
-    try {
-      if (localStorage.getItem("override-prefers-color-scheme")) box.checked = true;
-      box.addEventListener("change", function () {
-        if (box.checked) localStorage.setItem("override-prefers-color-scheme", 1);
-        else localStorage.removeItem("override-prefers-color-scheme");
-      });
-    } catch (e) { /* storage unavailable: the toggle still works per page */ }
-  })();
-<% end %>
diff --git a/app/views/layouts/admin.html.erb b/app/views/layouts/admin.html.erb
index 6ac9b756..04cc40ac 100644
--- a/app/views/layouts/admin.html.erb
+++ b/app/views/layouts/admin.html.erb
@@ -26,8 +26,7 @@
     <div id="wrapper">
       <div id="navigation">
         <div id="main_navigation">
-          <%= render :partial => 'admin/menu' if current_user %>
-          <%= render :partial => 'layouts/nav_switchers' %>
+          <%= render :partial => 'layouts/nav_controls' %>
         </div>
         <div id="search_widget" style="display: none;">
           <div>