From 6a230b767718d13374590475073404fc19cb418f Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Fri, 24 Oct 2025 11:40:27 +0000
Subject: [PATCH] fix: change Docker icon in left sidebar to Container icon

PROPERLY fix the Docker icon in the main left sidebar navigation by updating the DockerIcon component to use lucide's Container icon instead of Simple Icons Docker logo.

Also revert incorrect changes that added Docker to the Proxmox sub-navigation tabs - Docker should not be in the Proxmox settings sub-menu.
---
 .../src/components/Settings/Settings.tsx            |  4 ++--
 .../src/components/Settings/SettingsSectionNav.tsx  |  6 ------
 frontend-modern/src/components/icons/DockerIcon.tsx | 13 ++-----------
 3 files changed, 4 insertions(+), 19 deletions(-)

diff --git a/frontend-modern/src/components/Settings/Settings.tsx b/frontend-modern/src/components/Settings/Settings.tsx
index 057990d..cedd654 100644
--- a/frontend-modern/src/components/Settings/Settings.tsx
+++ b/frontend-modern/src/components/Settings/Settings.tsx
@@ -2098,9 +2098,9 @@ const Settings: Component<SettingsProps> = (props) => {
             </Show>
 
             <div class="p-6 lg:p-8">
-                <Show when={activeTab() === 'proxmox' || activeTab() === 'docker'}>
+                <Show when={activeTab() === 'proxmox'}>
                   <SettingsSectionNav
-                    current={activeTab() === 'docker' ? 'docker' : selectedAgent()}
+                    current={selectedAgent()}
                     onSelect={handleSelectAgent}
                     class="mb-6"
                   />
diff --git a/frontend-modern/src/components/Settings/SettingsSectionNav.tsx b/frontend-modern/src/components/Settings/SettingsSectionNav.tsx
index 7e55f1e..9c6e5f5 100644
--- a/frontend-modern/src/components/Settings/SettingsSectionNav.tsx
+++ b/frontend-modern/src/components/Settings/SettingsSectionNav.tsx
@@ -3,7 +3,6 @@ import { For } from 'solid-js';
 import Server from 'lucide-solid/icons/server';
 import HardDrive from 'lucide-solid/icons/hard-drive';
 import Mail from 'lucide-solid/icons/mail';
-import Container from 'lucide-solid/icons/container';
 
 type SettingsSection = 'pve' | 'pbs' | 'pmg' | 'docker' | 'host' | 'podman' | 'kubernetes';
 
@@ -33,11 +32,6 @@ const allSections: Array<{
     label: 'Mail Gateway',
     icon: Mail,
   },
-  {
-    id: 'docker',
-    label: 'Docker',
-    icon: Container,
-  },
 ];
 
 export const SettingsSectionNav: Component<SettingsSectionNavProps> = (props) => {
diff --git a/frontend-modern/src/components/icons/DockerIcon.tsx b/frontend-modern/src/components/icons/DockerIcon.tsx
index cadd9a8..627d5e5 100644
--- a/frontend-modern/src/components/icons/DockerIcon.tsx
+++ b/frontend-modern/src/components/icons/DockerIcon.tsx
@@ -1,5 +1,5 @@
 import type { Component } from 'solid-js';
-import { siDocker } from 'simple-icons';
+import Container from 'lucide-solid/icons/container';
 
 interface DockerIconProps {
   class?: string;
@@ -7,14 +7,5 @@ interface DockerIconProps {
 }
 
 export const DockerIcon: Component<DockerIconProps> = (props) => (
-  <svg
-    class={props.class}
-    role="img"
-    viewBox="0 0 24 24"
-    xmlns="http://www.w3.org/2000/svg"
-    aria-label={props.title ?? 'Docker'}
-  >
-    <title>{props.title ?? 'Docker'}</title>
-    <path d={siDocker.path} fill="currentColor" />
-  </svg>
+  <Container class={props.class} aria-label={props.title ?? 'Docker'} />
 );