From 99d9b572400fb071a1fc332b57bbd9f669ca445c Mon Sep 17 00:00:00 2001
From: Henry Jameson <me@hjkos.com>
Date: Tue, 5 May 2026 21:05:56 +0300
Subject: [PATCH] fix wide logo overlap issues

---
 src/components/desktop_nav/desktop_nav.scss | 17 ++---
 src/components/desktop_nav/desktop_nav.vue  | 78 +++++++++++----------
 src/components/search_bar/search_bar.vue    | 10 +--
 3 files changed, 48 insertions(+), 57 deletions(-)

diff --git a/src/components/desktop_nav/desktop_nav.scss b/src/components/desktop_nav/desktop_nav.scss
index 35a57d9fe..9755ac590 100644
--- a/src/components/desktop_nav/desktop_nav.scss
+++ b/src/components/desktop_nav/desktop_nav.scss
@@ -9,7 +9,7 @@
   .inner-nav {
     display: grid;
     grid-template-rows: var(--navbar-height);
-    grid-template-columns: 2fr auto 2fr;
+    grid-template-columns: minmax(5em, 1fr) auto minmax(5em, 1fr);
     grid-template-areas: "sitename logo actions";
     box-sizing: border-box;
     padding: 0 1.2em;
@@ -31,7 +31,7 @@
   }
 
   &.-logoLeft .inner-nav {
-    grid-template-columns: auto 2fr 2fr;
+    grid-template-columns: auto minmax(5em, 1fr) minmax(5em, 1fr);
     grid-template-areas: "logo sitename actions";
   }
 
@@ -92,23 +92,18 @@
 
   .actions {
     grid-area: actions;
+    justify-content: flex-end;
+    text-align: right;
+    z-index: 1;
   }
 
   .item {
-    flex: 1;
     line-height: var(--navbar-height);
     height: var(--navbar-height);
-    overflow: hidden;
     display: flex;
-    flex-wrap: wrap;
-
-    &.right {
-      justify-content: flex-end;
-      text-align: right;
-    }
   }
 
   .spacer {
-    width: 1em;
+    min-width: 1em;
   }
 }
diff --git a/src/components/desktop_nav/desktop_nav.vue b/src/components/desktop_nav/desktop_nav.vue
index a75f8506d..8c7f589b6 100644
--- a/src/components/desktop_nav/desktop_nav.vue
+++ b/src/components/desktop_nav/desktop_nav.vue
@@ -32,48 +32,50 @@
         >
       </router-link>
       <div class="item right actions">
-        <search-bar
+        <SearchBar
           v-if="currentUser || !privateMode"
           @toggled="onSearchBarToggled"
           @click.stop
         />
-        <button
-          class="button-unstyled nav-icon"
-          :title="$t('nav.preferences')"
-          @click.stop="openSettingsModal('user')"
-        >
-          <FAIcon
-            fixed-width
-            class="fa-scale-110 fa-old-padding"
-            icon="cog"
-          />
-        </button>
-        <button
-          v-if="currentUser && currentUser.role === 'admin'"
-          class="button-unstyled nav-icon"
-          target="_blank"
-          :title="$t('nav.administration')"
-          @click.stop="openSettingsModal('admin')"
-        >
-          <FAIcon
-            fixed-width
-            class="fa-scale-110 fa-old-padding"
-            icon="tachometer-alt"
-          />
-        </button>
-        <span class="spacer" />
-        <button
-          v-if="currentUser"
-          class="button-unstyled nav-icon"
-          :title="$t('login.logout')"
-          @click.stop.prevent="logout"
-        >
-          <FAIcon
-            fixed-width
-            class="fa-scale-110 fa-old-padding"
-            icon="sign-out-alt"
-          />
-        </button>
+        <template v-if="searchBarHidden">
+          <button
+            class="button-unstyled nav-icon"
+            :title="$t('nav.preferences')"
+            @click.stop="openSettingsModal('user')"
+          >
+            <FAIcon
+              fixed-width
+              class="fa-scale-110 fa-old-padding"
+              icon="cog"
+            />
+          </button>
+          <button
+            v-if="currentUser && currentUser.role === 'admin'"
+            class="button-unstyled nav-icon"
+            target="_blank"
+            :title="$t('nav.administration')"
+            @click.stop="openSettingsModal('admin')"
+          >
+            <FAIcon
+              fixed-width
+              class="fa-scale-110 fa-old-padding"
+              icon="tachometer-alt"
+            />
+          </button>
+          <span class="spacer" />
+          <button
+            v-if="currentUser"
+            class="button-unstyled nav-icon"
+            :title="$t('login.logout')"
+            @click.stop.prevent="logout"
+          >
+            <FAIcon
+              fixed-width
+              class="fa-scale-110 fa-old-padding"
+              icon="sign-out-alt"
+            />
+          </button>
+        </template>
       </div>
     </div>
     <teleport to="#modal">
diff --git a/src/components/search_bar/search_bar.vue b/src/components/search_bar/search_bar.vue
index d06b4e77d..9a9c971fb 100644
--- a/src/components/search_bar/search_bar.vue
+++ b/src/components/search_bar/search_bar.vue
@@ -51,8 +51,6 @@
           class="cancel-icon fa-scale-110 fa-old-padding"
         />
       </button>
-      <span class="spacer" />
-      <span class="spacer" />
     </template>
   </div>
 </template>
@@ -61,18 +59,14 @@
 
 <style lang="scss">
 .SearchBar {
-  display: inline-flex;
+  display: flex;
   align-items: baseline;
   vertical-align: baseline;
   justify-content: flex-end;
 
-  &.-expanded {
-    width: 100%;
-  }
-
   .search-bar-input,
   .search-button {
-    height: 29px;
+    height: 2em;
   }
 
   .search-bar-input {