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.
This commit is contained in:
rcourtman 2025-10-24 11:40:27 +00:00
parent 78d1689b34
commit 6a230b7677
3 changed files with 4 additions and 19 deletions

View file

@ -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"
/>

View file

@ -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) => {

View file

@ -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'} />
);