From 714dc718f22e536c93ef433edef714ae4829180c Mon Sep 17 00:00:00 2001
From: curo1305 <curo1305@proton.me>
Date: Fri, 17 Apr 2026 18:30:05 +0200
Subject: [PATCH] Remove 'All documents' sub-item; Documents label now links to
 /apps/documents

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/components/Sidebar.tsx | 44 +++++++++++++++--------------
 1 file changed, 23 insertions(+), 21 deletions(-)

diff --git a/frontend/src/components/Sidebar.tsx b/frontend/src/components/Sidebar.tsx
index 5221ba8..0763835 100644
--- a/frontend/src/components/Sidebar.tsx
+++ b/frontend/src/components/Sidebar.tsx
@@ -140,36 +140,38 @@ export default function Sidebar() {
             <div className="mt-0.5 space-y-0.5">
               {/* Documents service */}
               <div>
-                <button
-                  onClick={() => setDocsOpen((o) => !o)}
+                <div
                   className={cn(
-                    "w-full flex items-center rounded-lg transition-colors text-sm",
-                    "pl-8 pr-3 py-1.5 gap-2",
+                    "flex items-center rounded-lg transition-colors text-sm",
                     isDocsRoute
                       ? "bg-primary/10 text-primary"
                       : "text-muted hover:bg-muted/20 hover:text-foreground"
                   )}
                 >
-                  <FileText className="h-4 w-4 shrink-0" />
-                  <span className="flex-1 text-left whitespace-nowrap">Documents</span>
-                  {docsOpen ? (
-                    <ChevronDown className="h-3.5 w-3.5 shrink-0" />
-                  ) : (
-                    <ChevronRight className="h-3.5 w-3.5 shrink-0" />
-                  )}
-                </button>
+                  <NavLink
+                    to="/apps/documents"
+                    end
+                    className="flex items-center gap-2 pl-8 pr-2 py-1.5 flex-1 min-w-0"
+                  >
+                    <FileText className="h-4 w-4 shrink-0" />
+                    <span className="whitespace-nowrap">Documents</span>
+                  </NavLink>
+                  <button
+                    onClick={() => setDocsOpen((o) => !o)}
+                    className="px-2 py-1.5 rounded-r-lg"
+                    aria-label={docsOpen ? "Collapse documents" : "Expand documents"}
+                  >
+                    {docsOpen ? (
+                      <ChevronDown className="h-3.5 w-3.5 shrink-0" />
+                    ) : (
+                      <ChevronRight className="h-3.5 w-3.5 shrink-0" />
+                    )}
+                  </button>
+                </div>
 
-                {/* Documents open link + categories */}
+                {/* Categories */}
                 {docsOpen && (
                   <div className="mt-0.5 space-y-0.5">
-                    <NavLink
-                      to="/apps/documents"
-                      end
-                      className={({ isActive }) => subSubItemClass(isActive)}
-                    >
-                      <span className="whitespace-nowrap">All documents</span>
-                    </NavLink>
-
                     {categories.map((cat) => (
                       <NavLink
                         key={cat.id}