From e6372d3937641add8dac15a22ebd576909e1bb8b Mon Sep 17 00:00:00 2001
From: shamoon <4887959+shamoon@users.noreply.github.com>
Date: Tue, 15 Sep 2026 07:57:36 -0700
Subject: [PATCH] Make share links also hideable

---
 .../app/components/admin/settings/settings.component.ts    | 1 +
 .../src/app/components/app-frame/app-frame.component.html  | 7 +++++--
 .../app/components/app-frame/app-frame.component.spec.ts   | 5 +++++
 src-ui/src/app/data/ui-settings.ts                         | 1 +
 4 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/src-ui/src/app/components/admin/settings/settings.component.ts b/src-ui/src/app/components/admin/settings/settings.component.ts
index 9076d2a3e..a4ead8373 100644
--- a/src-ui/src/app/components/admin/settings/settings.component.ts
+++ b/src-ui/src/app/components/admin/settings/settings.component.ts
@@ -110,6 +110,7 @@ const documentDetailFieldOptions = [
 const sidebarItemLabels: Record<HideableSidebarItemID, string> = {
   [HideableSidebarItemID.Dashboard]: $localize`Dashboard`,
   [HideableSidebarItemID.SavedViews]: $localize`Saved Views`,
+  [HideableSidebarItemID.ShareLinks]: $localize`Share Links`,
   [HideableSidebarItemID.Workflows]: $localize`Workflows`,
   [HideableSidebarItemID.Mail]: $localize`Mail`,
   [HideableSidebarItemID.Documentation]: $localize`Documentation`,
diff --git a/src-ui/src/app/components/app-frame/app-frame.component.html b/src-ui/src/app/components/app-frame/app-frame.component.html
index bafc34cd3..942c6a195 100644
--- a/src-ui/src/app/components/app-frame/app-frame.component.html
+++ b/src-ui/src/app/components/app-frame/app-frame.component.html
@@ -251,12 +251,15 @@
               }
             </li>
             @if (canManageShareLinks) {
-              <li class="nav-item app-link">
-                <a class="nav-link" routerLink="share-links" routerLinkActive="active" (click)="closeMenu()"
+              <li class="nav-item app-link position-relative" [class.d-none]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.ShareLinks) && !settingsService.organizingSidebarItems()">
+                <a class="nav-link" [class.opacity-50]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.ShareLinks)" [class.pe-5]="settingsService.organizingSidebarItems() && !slimSidebarEnabled && !slimSidebarAnimating()" routerLink="share-links" routerLinkActive="active" (click)="closeMenu()"
                   ngbPopover="Share links" i18n-ngbPopover [disablePopover]="!slimSidebarPopoversEnabled" placement="end"
                   container="body" triggers="mouseenter:mouseleave" popoverClass="popover-slim">
                   <i-bs class="me-2" name="link"></i-bs><span class="nav-link-label"><ng-container i18n>Share links</ng-container></span>
                 </a>
+                @if (settingsService.organizingSidebarItems()) {
+                  <pngx-input-switch class="position-absolute top-50 end-0 translate-middle-y me-1" [class.d-none]="slimSidebarEnabled || slimSidebarAnimating()" [compact]="true" title="Share Links" i18n-title [ngModel]="!settingsService.sidebarItemIsHidden(HideableSidebarItemID.ShareLinks)" (ngModelChange)="toggleSidebarItem(HideableSidebarItemID.ShareLinks, $event)"></pngx-input-switch>
+                }
               </li>
             }
             <li class="nav-item app-link position-relative" [class.d-none]="settingsService.sidebarItemIsHidden(HideableSidebarItemID.Workflows) && !settingsService.organizingSidebarItems()"
diff --git a/src-ui/src/app/components/app-frame/app-frame.component.spec.ts b/src-ui/src/app/components/app-frame/app-frame.component.spec.ts
index 4dd88c84d..afc8b095b 100644
--- a/src-ui/src/app/components/app-frame/app-frame.component.spec.ts
+++ b/src-ui/src/app/components/app-frame/app-frame.component.spec.ts
@@ -291,6 +291,7 @@ describe('AppFrameComponent', () => {
     settingsService.set(SETTINGS_KEYS.SIDEBAR_HIDDEN_ITEMS, [
       HideableSidebarItemID.Dashboard,
       HideableSidebarItemID.Workflows,
+      HideableSidebarItemID.ShareLinks,
     ])
     fixture.detectChanges()
 
@@ -302,6 +303,10 @@ describe('AppFrameComponent', () => {
       fixture.nativeElement.querySelector('[routerLink="workflows"]')
         .parentElement.classList
     ).toContain('d-none')
+    expect(
+      fixture.nativeElement.querySelector('[routerLink="share-links"]')
+        .parentElement.classList
+    ).toContain('d-none')
     expect(
       fixture.nativeElement.querySelector('[routerLink="mail"]').parentElement
         .classList
diff --git a/src-ui/src/app/data/ui-settings.ts b/src-ui/src/app/data/ui-settings.ts
index a6d28669f..5bc500872 100644
--- a/src-ui/src/app/data/ui-settings.ts
+++ b/src-ui/src/app/data/ui-settings.ts
@@ -27,6 +27,7 @@ export enum CollapsibleSection {
 export enum HideableSidebarItemID {
   Dashboard = 'dashboard',
   SavedViews = 'saved_views',
+  ShareLinks = 'share_links',
   Workflows = 'workflows',
   Mail = 'mail',
   Documentation = 'documentation',