From cb4175360587903505b5703f24df966dcc1ba6ff Mon Sep 17 00:00:00 2001
From: curo1305 <curo1305@proton.me>
Date: Tue, 16 Jun 2026 08:35:02 +0200
Subject: [PATCH] refactor(10-12): migrate sidebar + layout + TreeItem SVGs to
 AppIcon

- StorageBrowser.vue: replace 8 inline SVGs (plus, folder x2, pencil, trash, share, folderMove, document)
- AppSidebar.vue: replace 8 inline SVGs (tag, inbox, chevronRight x2, folder, cloud, shield, cog dual-path, logout)
- AdminSidebar.vue: replace 6 inline SVGs (home, users, chartBar, lightBulb, clipboardList, logout)
- TreeItem.vue: replace 1 inline SVG (chevronRight expand/collapse)
---
 .../src/components/admin/AdminSidebar.vue     | 31 +++--------
 frontend/src/components/layout/AppSidebar.vue | 53 +++++--------------
 .../src/components/storage/StorageBrowser.vue | 45 ++++------------
 frontend/src/components/ui/TreeItem.vue       | 11 ++--
 4 files changed, 35 insertions(+), 105 deletions(-)

diff --git a/frontend/src/components/admin/AdminSidebar.vue b/frontend/src/components/admin/AdminSidebar.vue
index e147f8e..1a8c040 100644
--- a/frontend/src/components/admin/AdminSidebar.vue
+++ b/frontend/src/components/admin/AdminSidebar.vue
@@ -11,10 +11,7 @@
         class="nav-link"
         :class="{ 'nav-link-active': $route.path === '/admin' }"
       >
-        <svg class="w-4 h-4 mr-2 shrink-0" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-            d="M3 12l2-2m0 0l7-7 7 7M5 10v10a1 1 0 001 1h3m10-11l2 2m-2-2v10a1 1 0 01-1 1h-3m-6 0a1 1 0 001-1v-4a1 1 0 011-1h2a1 1 0 011 1v4a1 1 0 001 1m-6 0h6" />
-        </svg>
+        <AppIcon name="home" class="w-4 h-4 mr-2 shrink-0" />
         Overview
       </router-link>
 
@@ -23,10 +20,7 @@
         class="nav-link"
         :class="{ 'nav-link-active': $route.path.startsWith('/admin/users') }"
       >
-        <svg class="w-4 h-4 mr-2 shrink-0" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-            d="M12 4.354a4 4 0 110 5.292M15 21H3v-1a6 6 0 0112 0v1zm0 0h6v-1a6 6 0 00-9-5.197M13 7a4 4 0 11-8 0 4 4 0 018 0z" />
-        </svg>
+        <AppIcon name="users" class="w-4 h-4 mr-2 shrink-0" />
         Users
       </router-link>
 
@@ -35,10 +29,7 @@
         class="nav-link"
         :class="{ 'nav-link-active': $route.path.startsWith('/admin/quotas') }"
       >
-        <svg class="w-4 h-4 mr-2 shrink-0" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-            d="M9 19v-6a2 2 0 00-2-2H5a2 2 0 00-2 2v6a2 2 0 002 2h2a2 2 0 002-2zm0 0V9a2 2 0 012-2h2a2 2 0 012 2v10m-6 0a2 2 0 002 2h2a2 2 0 002-2m0 0V5a2 2 0 012-2h2a2 2 0 012 2v14a2 2 0 01-2 2h-2a2 2 0 01-2-2z" />
-        </svg>
+        <AppIcon name="chartBar" class="w-4 h-4 mr-2 shrink-0" />
         Quotas
       </router-link>
 
@@ -47,10 +38,7 @@
         class="nav-link"
         :class="{ 'nav-link-active': $route.path.startsWith('/admin/ai') }"
       >
-        <svg class="w-4 h-4 mr-2 shrink-0" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-            d="M9.663 17h4.673M12 3v1m6.364 1.636l-.707.707M21 12h-1M4 12H3m3.343-5.657l-.707-.707m2.828 9.9a5 5 0 117.072 0l-.548.547A3.374 3.374 0 0014 18.469V19a2 2 0 11-4 0v-.531c0-.895-.356-1.754-.988-2.386l-.548-.547z" />
-        </svg>
+        <AppIcon name="lightBulb" class="w-4 h-4 mr-2 shrink-0" />
         AI Config
       </router-link>
 
@@ -59,10 +47,7 @@
         class="nav-link"
         :class="{ 'nav-link-active': $route.path.startsWith('/admin/audit') }"
       >
-        <svg class="w-4 h-4 mr-2 shrink-0" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-            d="M9 5H7a2 2 0 00-2 2v12a2 2 0 002 2h10a2 2 0 002-2V7a2 2 0 00-2-2h-2M9 5a2 2 0 002 2h2a2 2 0 002-2M9 5a2 2 0 012-2h2a2 2 0 012 2m-3 7h3m-3 4h3m-6-4h.01M9 16h.01" />
-        </svg>
+        <AppIcon name="clipboardList" class="w-4 h-4 mr-2 shrink-0" />
         Audit Log
       </router-link>
     </nav>
@@ -78,10 +63,7 @@
           aria-label="Sign out"
           class="text-gray-400 hover:text-gray-600 transition-colors"
         >
-          <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-            <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-              d="M17 16l4-4m0 0l-4-4m4 4H7m6 4v1a3 3 0 01-3 3H6a3 3 0 01-3-3V7a3 3 0 013-3h4a3 3 0 013 3v1" />
-          </svg>
+          <AppIcon name="logout" class="w-4 h-4" />
         </button>
       </div>
     </div>
@@ -91,6 +73,7 @@
 <script setup>
 import { useRouter } from 'vue-router'
 import { useAuthStore } from '../../stores/auth.js'
+import AppIcon from '../ui/AppIcon.vue'
 
 const authStore = useAuthStore()
 const router = useRouter()
diff --git a/frontend/src/components/layout/AppSidebar.vue b/frontend/src/components/layout/AppSidebar.vue
index d0d69ac..e5fc451 100644
--- a/frontend/src/components/layout/AppSidebar.vue
+++ b/frontend/src/components/layout/AppSidebar.vue
@@ -11,10 +11,7 @@
         class="nav-link"
         :class="{ 'nav-link-active': $route.path.startsWith('/topics') }"
       >
-        <svg class="w-4 h-4 mr-2 shrink-0" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-            d="M7 7h.01M7 3h5c.512 0 1.024.195 1.414.586l7 7a2 2 0 010 2.828l-7 7a2 2 0 01-2.828 0l-7-7A1.994 1.994 0 013 12V7a4 4 0 014-4z" />
-        </svg>
+        <AppIcon name="tag" class="w-4 h-4 mr-2 shrink-0" />
         All Topics
       </router-link>
 
@@ -24,10 +21,7 @@
         :class="{ 'nav-link-active': $route.path === '/shared' }"
       >
         <div class="w-4 h-4 mr-2 shrink-0 rounded bg-purple-50 flex items-center justify-center">
-          <svg class="w-3 h-3 text-purple-500" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-            <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-              d="M20 13V6a2 2 0 00-2-2H6a2 2 0 00-2 2v7m16 0v5a2 2 0 01-2 2H6a2 2 0 01-2-2v-5m16 0h-2.586a1 1 0 00-.707.293l-2.414 2.414a1 1 0 01-.707.293h-3.172a1 1 0 01-.707-.293l-2.414-2.414A1 1 0 006.586 13H4" />
-          </svg>
+          <AppIcon name="inbox" class="w-3 h-3 text-purple-500" />
         </div>
         <span class="flex-1">Shared with me</span>
         <span
@@ -45,13 +39,11 @@
             class="p-1 rounded hover:bg-gray-100 text-gray-400 hover:text-gray-600 transition-colors shrink-0"
             :title="foldersExpanded ? 'Collapse folders' : 'Expand folders'"
           >
-            <svg
+            <AppIcon
+              name="chevronRight"
               class="w-3 h-3 transition-transform duration-150"
               :class="foldersExpanded ? 'rotate-90' : ''"
-              fill="none" stroke="currentColor" viewBox="0 0 24 24"
-            >
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M9 5l7 7-7 7" />
-            </svg>
+            />
           </button>
 
           <router-link
@@ -59,10 +51,7 @@
             class="nav-link flex-1 min-w-0"
             :class="{ 'nav-link-active': $route.path === '/' || $route.path.startsWith('/folders/') }"
           >
-            <svg class="w-4 h-4 mr-2 shrink-0 text-amber-500" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                d="M3 7a2 2 0 012-2h4l2 2h8a2 2 0 012 2v9a2 2 0 01-2 2H5a2 2 0 01-2-2V7z" />
-            </svg>
+            <AppIcon name="folder" class="w-4 h-4 mr-2 shrink-0 text-amber-500" />
             Folders
           </router-link>
         </div>
@@ -97,13 +86,11 @@
             class="p-1 rounded hover:bg-gray-100 text-gray-400 hover:text-gray-600 transition-colors shrink-0"
             :title="cloudExpanded ? 'Collapse cloud storage' : 'Expand cloud storage'"
           >
-            <svg
+            <AppIcon
+              name="chevronRight"
               class="w-3 h-3 transition-transform duration-150"
               :class="cloudExpanded ? 'rotate-90' : ''"
-              fill="none" stroke="currentColor" viewBox="0 0 24 24"
-            >
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M9 5l7 7-7 7" />
-            </svg>
+            />
           </button>
 
           <router-link
@@ -111,10 +98,7 @@
             class="nav-link flex-1 min-w-0"
             :class="{ 'nav-link-active': $route.path.startsWith('/cloud') }"
           >
-            <svg class="w-4 h-4 mr-2 shrink-0 text-sky-500" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                d="M3 15a4 4 0 004 4h9a5 5 0 10-.1-9.999 5.002 5.002 0 10-9.78 2.096A4.001 4.001 0 003 15z" />
-            </svg>
+            <AppIcon name="cloud" class="w-4 h-4 mr-2 shrink-0 text-sky-500" />
             Cloud Storage
           </router-link>
         </div>
@@ -187,10 +171,7 @@
         class="nav-link"
         :class="{ 'nav-link-active': $route.path === '/admin' }"
       >
-        <svg class="w-4 h-4 mr-2 shrink-0" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-            d="M9 12l2 2 4-4m5.618-4.016A11.955 11.955 0 0112 2.944a11.955 11.955 0 01-8.618 3.04A12.02 12.02 0 003 9c0 5.591 3.824 10.29 9 11.622 5.176-1.332 9-6.03 9-11.622 0-1.042-.133-2.052-.382-3.016z" />
-        </svg>
+        <AppIcon name="shield" class="w-4 h-4 mr-2 shrink-0" />
         Admin
       </router-link>
 
@@ -199,11 +180,7 @@
         class="nav-link"
         :class="{ 'nav-link-active': $route.path === '/settings' }"
       >
-        <svg class="w-4 h-4 mr-2 shrink-0" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-            d="M10.325 4.317c.426-1.756 2.924-1.756 3.35 0a1.724 1.724 0 002.573 1.066c1.543-.94 3.31.826 2.37 2.37a1.724 1.724 0 001.065 2.572c1.756.426 1.756 2.924 0 3.35a1.724 1.724 0 00-1.066 2.573c.94 1.543-.826 3.31-2.37 2.37a1.724 1.724 0 00-2.572 1.065c-.426 1.756-2.924 1.756-3.35 0a1.724 1.724 0 00-2.573-1.066c-1.543.94-3.31-.826-2.37-2.37a1.724 1.724 0 00-1.065-2.572c-1.756-.426-1.756-2.924 0-3.35a1.724 1.724 0 001.066-2.573c-.94-1.543.826-3.31 2.37-2.37.996.608 2.296.07 2.572-1.065z" />
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M15 12a3 3 0 11-6 0 3 3 0 016 0z" />
-        </svg>
+        <AppIcon name="cog" class="w-4 h-4 mr-2 shrink-0" />
         Settings
       </router-link>
 
@@ -217,10 +194,7 @@
           aria-label="Sign out"
           class="text-gray-400 hover:text-gray-600 transition-colors"
         >
-          <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-            <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-              d="M17 16l4-4m0 0l-4-4m4 4H7m6 4v1a3 3 0 01-3 3H6a3 3 0 01-3-3V7a3 3 0 013-3h4a3 3 0 013 3v1" />
-          </svg>
+          <AppIcon name="logout" class="w-4 h-4" />
         </button>
       </div>
     </div>
@@ -230,6 +204,7 @@
 <script setup>
 import { ref, computed, onMounted, watch } from 'vue'
 import { useRouter } from 'vue-router'
+import AppIcon from '../ui/AppIcon.vue'
 import { useTopicsStore } from '../../stores/topics.js'
 import { useAuthStore } from '../../stores/auth.js'
 import { useFoldersStore } from '../../stores/folders.js'
diff --git a/frontend/src/components/storage/StorageBrowser.vue b/frontend/src/components/storage/StorageBrowser.vue
index d22cc7d..d99f63c 100644
--- a/frontend/src/components/storage/StorageBrowser.vue
+++ b/frontend/src/components/storage/StorageBrowser.vue
@@ -22,9 +22,7 @@
             @click="$emit('new-folder')"
             class="flex items-center gap-1.5 px-3 py-1.5 text-sm font-medium text-indigo-600 border border-indigo-200 hover:bg-indigo-50 rounded-lg transition-colors"
           >
-            <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M12 4v16m8-8H4" />
-            </svg>
+            <AppIcon name="plus" class="w-4 h-4" />
             New folder
           </button>
         </div>
@@ -52,10 +50,7 @@
       <!-- Inline new-folder row (local only) -->
       <div v-if="showNewFolderInput" class="mx-6 mt-1 px-4 py-2.5 grid grid-cols-[2rem_1fr_6rem_8rem_6rem] gap-3 items-center rounded-lg border border-amber-200 bg-amber-50/40">
         <div class="w-7 h-7 bg-amber-50 rounded-lg flex items-center justify-center">
-          <svg class="w-4 h-4 text-amber-400" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-            <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-              d="M3 7a2 2 0 012-2h4l2 2h8a2 2 0 012 2v9a2 2 0 01-2 2H5a2 2 0 01-2-2V7z" />
-          </svg>
+          <AppIcon name="folder" class="w-4 h-4 text-amber-400" />
         </div>
         <div class="flex items-center gap-2 col-span-4">
           <input
@@ -91,10 +86,7 @@
           @drop.prevent="mode === 'local' ? onDropDocOnFolder(folder.id) : null"
         >
           <div class="w-7 h-7 bg-amber-50 rounded-lg flex items-center justify-center shrink-0">
-            <svg class="w-4 h-4 text-amber-500" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                d="M3 7a2 2 0 012-2h4l2 2h8a2 2 0 012 2v9a2 2 0 01-2 2H5a2 2 0 01-2-2V7z" />
-            </svg>
+            <AppIcon name="folder" class="w-4 h-4 text-amber-500" />
           </div>
 
           <!-- Name / rename input -->
@@ -118,17 +110,11 @@
             <template v-if="mode === 'local'">
               <button @click.stop="startRename(folder)" title="Rename"
                 class="p-1.5 rounded hover:bg-gray-200 text-gray-400 hover:text-gray-700 transition-colors">
-                <svg class="w-3.5 h-3.5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                  <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                    d="M11 5H6a2 2 0 00-2 2v11a2 2 0 002 2h11a2 2 0 002-2v-5m-1.414-9.414a2 2 0 112.828 2.828L11.828 15H9v-2.828l8.586-8.586z" />
-                </svg>
+                <AppIcon name="pencil" class="w-3.5 h-3.5" />
               </button>
               <button @click.stop="$emit('folder-delete', folder)" title="Delete folder"
                 class="p-1.5 rounded hover:bg-red-50 text-gray-400 hover:text-red-500 transition-colors">
-                <svg class="w-3.5 h-3.5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                  <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                    d="M19 7l-.867 12.142A2 2 0 0116.138 21H7.862a2 2 0 01-1.995-1.858L5 7m5 4v6m4-6v6m1-10V4a1 1 0 00-1-1h-4a1 1 0 00-1 1v3M4 7h16" />
-                </svg>
+                <AppIcon name="trash" class="w-3.5 h-3.5" />
               </button>
             </template>
           </div>
@@ -146,10 +132,7 @@
           @dragend="onFileDragEnd"
         >
           <div class="w-7 h-7 bg-indigo-50 rounded-lg flex items-center justify-center shrink-0">
-            <svg class="w-4 h-4 text-indigo-400" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                d="M9 12h6m-6 4h6m2 5H7a2 2 0 01-2-2V5a2 2 0 012-2h5.586a1 1 0 01.707.293l5.414 5.414a1 1 0 01.293.707V19a2 2 0 01-2 2z" />
-            </svg>
+            <AppIcon name="document" class="w-4 h-4 text-indigo-400" />
           </div>
 
           <!-- Name + topics + shared badge -->
@@ -177,10 +160,7 @@
               <!-- Share -->
               <button @click.stop="$emit('file-share', file)" title="Share"
                 class="p-1.5 rounded hover:bg-gray-200 text-gray-400 hover:text-gray-700 transition-colors">
-                <svg class="w-3.5 h-3.5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                  <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                    d="M8.684 13.342C8.886 12.938 9 12.482 9 12c0-.482-.114-.938-.316-1.342m0 2.684a3 3 0 110-2.684m0 2.684l6.632 3.316m-6.632-6l6.632-3.316m0 0a3 3 0 105.367-2.684 3 3 0 00-5.367 2.684zm0 9.316a3 3 0 105.368 2.684 3 3 0 00-5.368-2.684z" />
-                </svg>
+                <AppIcon name="share" class="w-3.5 h-3.5" />
               </button>
               <!-- Move -->
               <div class="relative">
@@ -189,19 +169,13 @@
                   title="Move to folder"
                   class="p-1.5 rounded hover:bg-gray-200 text-gray-400 hover:text-gray-700 transition-colors"
                 >
-                  <svg class="w-3.5 h-3.5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                    <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                      d="M3 7a2 2 0 012-2h4l2 2h8a2 2 0 012 2v8a2 2 0 01-2 2H5a2 2 0 01-2-2V7z" />
-                  </svg>
+                  <AppIcon name="folderMove" class="w-3.5 h-3.5" />
                 </button>
               </div>
               <!-- Delete -->
               <button @click.stop="$emit('file-delete', file.id)" title="Delete"
                 class="p-1.5 rounded hover:bg-red-50 text-gray-400 hover:text-red-500 transition-colors">
-                <svg class="w-3.5 h-3.5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                  <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
-                    d="M19 7l-.867 12.142A2 2 0 0116.138 21H7.862a2 2 0 01-1.995-1.858L5 7m5 4v6m4-6v6m1-10V4a1 1 0 00-1-1h-4a1 1 0 00-1 1v3M4 7h16" />
-                </svg>
+                <AppIcon name="trash" class="w-3.5 h-3.5" />
               </button>
             </template>
           </div>
@@ -280,6 +254,7 @@
 
 <script setup>
 import { ref, computed, nextTick, onMounted, onUnmounted } from 'vue'
+import AppIcon from '../ui/AppIcon.vue'
 import BreadcrumbBar from '../ui/BreadcrumbBar.vue'
 import EmptyState from '../ui/EmptyState.vue'
 import SearchBar from '../documents/SearchBar.vue'
diff --git a/frontend/src/components/ui/TreeItem.vue b/frontend/src/components/ui/TreeItem.vue
index 580dca2..a5e5e4b 100644
--- a/frontend/src/components/ui/TreeItem.vue
+++ b/frontend/src/components/ui/TreeItem.vue
@@ -12,15 +12,11 @@
         class="w-5 h-5 flex items-center justify-center text-gray-400 hover:text-gray-600 shrink-0 transition-colors"
         :aria-label="expanded ? 'Collapse' : 'Expand'"
       >
-        <svg
+        <AppIcon
+          name="chevronRight"
           class="w-3 h-3 transition-transform duration-150"
           :class="{ 'rotate-90': expanded }"
-          fill="none"
-          stroke="currentColor"
-          viewBox="0 0 24 24"
-        >
-          <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2.5" d="M9 5l7 7-7 7" />
-        </svg>
+        />
       </button>
       <span v-else class="w-5 h-5 shrink-0"></span>
 
@@ -77,6 +73,7 @@
 
 <script setup>
 import { ref } from 'vue'
+import AppIcon from './AppIcon.vue'
 
 const props = defineProps({
   label: { type: String, required: true },