From 28c813762f56541f9fc845710483b5f6b96da10c Mon Sep 17 00:00:00 2001
From: jamesread <contact@jread.com>
Date: Thu, 6 Nov 2025 00:38:56 +0000
Subject: [PATCH] fix: #685 - show navigation (and picocrank upgrade)

---
 .gitignore                     |  2 ++
 frontend/package-lock.json     |  8 +++----
 frontend/package.json          |  2 +-
 frontend/resources/vue/App.vue | 40 +++++++++++++++++++---------------
 4 files changed, 29 insertions(+), 23 deletions(-)

diff --git a/.gitignore b/.gitignore
index 561630f..f2d12e3 100644
--- a/.gitignore
+++ b/.gitignore
@@ -17,3 +17,5 @@ webui/
 server.log
 OliveTin
 integration-tests/configs/authRequireGuestsToLogin/sessions.yaml
+webui
+webui.dev
diff --git a/frontend/package-lock.json b/frontend/package-lock.json
index 48f4f14..a8a3291 100644
--- a/frontend/package-lock.json
+++ b/frontend/package-lock.json
@@ -17,7 +17,7 @@
 				"@xterm/addon-fit": "^0.10.0",
 				"@xterm/xterm": "^5.5.0",
 				"iconify-icon": "^3.0.2",
-				"picocrank": "^1.6.4",
+				"picocrank": "^1.8.0",
 				"unplugin-vue-components": "^30.0.0",
 				"vite": "^7.1.12",
 				"vue-router": "^4.6.3"
@@ -2362,9 +2362,9 @@
 			"license": "ISC"
 		},
 		"node_modules/picocrank": {
-			"version": "1.6.4",
-			"resolved": "https://registry.npmjs.org/picocrank/-/picocrank-1.6.4.tgz",
-			"integrity": "sha512-zD1wnkoUDAXZOUs9zKqS4rqz9mljeqFwM7QWx4ykXJsmH6iOLAIKh2AVlxa384oeXJXIWM9VLiySEnhQZmQmjA==",
+			"version": "1.8.0",
+			"resolved": "https://registry.npmjs.org/picocrank/-/picocrank-1.8.0.tgz",
+			"integrity": "sha512-YPGmXvw7vvjIcgrAe3io87kZDM+NUa+aiEYxk8CVqBzgI4koXeF+2VEGPHBwknZBBEbJfXsSdnxVwXrLKpWKfw==",
 			"license": "ISC",
 			"dependencies": {
 				"@hugeicons/core-free-icons": "^1.0.16",
diff --git a/frontend/package.json b/frontend/package.json
index 357ee66..6e86726 100644
--- a/frontend/package.json
+++ b/frontend/package.json
@@ -30,7 +30,7 @@
 		"@xterm/addon-fit": "^0.10.0",
 		"@xterm/xterm": "^5.5.0",
 		"iconify-icon": "^3.0.2",
-		"picocrank": "^1.6.4",
+		"picocrank": "^1.8.0",
 		"unplugin-vue-components": "^30.0.0",
 		"vite": "^7.1.12",
 		"vue-router": "^4.6.3"
diff --git a/frontend/resources/vue/App.vue b/frontend/resources/vue/App.vue
index 2c18e31..8e552bf 100644
--- a/frontend/resources/vue/App.vue
+++ b/frontend/resources/vue/App.vue
@@ -1,5 +1,5 @@
 <template>
-    <Header title="OliveTin" :logoUrl="logoUrl" @toggleSidebar="toggleSidebar">
+    <Header title="OliveTin" :logoUrl="logoUrl" @toggleSidebar="toggleSidebar" :sidebarEnabled="showNavigation">
         <template #toolbar>
             <div id="banner" v-if="bannerMessage" :style="bannerCss">
                 <p>{{ bannerMessage }}</p>
@@ -86,7 +86,9 @@ const initError = ref(false)
 const initErrorMessage = ref('')
 
 function toggleSidebar() {
-    sidebar.value.toggle()
+    if (sidebar.value && showNavigation.value) {
+        sidebar.value.toggle()
+    }
 }
 
 function updateHeaderFromInit() {
@@ -132,25 +134,27 @@ async function requestInit() {
         showLogs.value = initResponse.showLogList
         showDiagnostics.value = initResponse.showDiagnostics
 
-        for (const rootDashboard of initResponse.rootDashboards) {
-            sidebar.value.addNavigationLink({
-                id: rootDashboard,
-                name: rootDashboard,
-                title: rootDashboard,
-                path: rootDashboard === 'Actions' ? '/' : `/dashboards/${rootDashboard}`,
-                icon: DashboardSquare01Icon,
-            })
-        }
+        if (showNavigation.value && sidebar.value) {
+            for (const rootDashboard of initResponse.rootDashboards) {
+                sidebar.value.addNavigationLink({
+                    id: rootDashboard,
+                    name: rootDashboard,
+                    title: rootDashboard,
+                    path: rootDashboard === 'Actions' ? '/' : `/dashboards/${rootDashboard}`,
+                    icon: DashboardSquare01Icon,
+                })
+            }
 
-        sidebar.value.addSeparator()
-        sidebar.value.addRouterLink('Entities')
+            sidebar.value.addSeparator()
+            sidebar.value.addRouterLink('Entities')
 
-        if (showLogs.value) {
-            sidebar.value.addRouterLink('Logs')
-        }
+            if (showLogs.value) {
+                sidebar.value.addRouterLink('Logs')
+            }
 
-        if (showDiagnostics.value) {
-            sidebar.value.addRouterLink('Diagnostics')
+            if (showDiagnostics.value) {
+                sidebar.value.addRouterLink('Diagnostics')
+            }
         }
 
         hasLoaded.value = true;