From 2b375685887de454b0880ed362f6841e47a4e1ef Mon Sep 17 00:00:00 2001 From: Neil <4138956+nwparker@users.noreply.github.com> Date: Fri, 19 Jun 2026 01:56:59 -0700 Subject: [PATCH] fix(sidebar): tidy Live Ports card header (#5790) Co-authored-by: Orca --- .../components/sidebar/WorktreeCardPorts.tsx | 37 ++++--------------- 1 file changed, 7 insertions(+), 30 deletions(-) diff --git a/src/renderer/src/components/sidebar/WorktreeCardPorts.tsx b/src/renderer/src/components/sidebar/WorktreeCardPorts.tsx index 9d79583cf..a8db673ff 100644 --- a/src/renderer/src/components/sidebar/WorktreeCardPorts.tsx +++ b/src/renderer/src/components/sidebar/WorktreeCardPorts.tsx @@ -1,5 +1,5 @@ import React, { useCallback, useMemo } from 'react' -import { Plug, Copy, ExternalLink, FolderOpen, Trash2 } from 'lucide-react' +import { Plug, Copy, ExternalLink, Trash2 } from 'lucide-react' import { toast } from 'sonner' import { useAppStore } from '@/store' import { HoverCard, HoverCardContent, HoverCardTrigger } from '@/components/ui/hover-card' @@ -11,7 +11,6 @@ import { getRuntimeEnvironmentIdForWorktree } from '@/lib/worktree-runtime-owner import { canStopWorkspacePort, getPortOpenBrowserTooltipLabel, - goToWorkspacePortOwner, killWorkspacePortForTarget, openWorkspacePortInBrowser, refreshWorkspacePortScanAfterStop, @@ -293,44 +292,22 @@ function WorktreePortRow({ port }: { port: WorkspacePort }): React.JSX.Element { export function WorktreeCardPortsDetails({ ports }: WorktreeCardPortsProps): React.JSX.Element | null { - const recordFeatureInteraction = useAppStore((s) => s.recordFeatureInteraction) - const handleGoToWorktree = useCallback( - (event: React.MouseEvent) => { - event.stopPropagation() - recordFeatureInteraction('ports') - const ownerPort = ports[0] - if (!ownerPort || !goToWorkspacePortOwner(ownerPort)) { - toast.error( - translate('auto.components.sidebar.WorktreeCardPorts.3e5f66564e', 'Workspace unavailable') - ) - } - }, - [ports, recordFeatureInteraction] - ) - if (ports.length === 0) { return null } return ( + {/* Count lives inline after the label (not on the right, which is the action zone); + the "Go to Worktree" action is omitted here since the card is already scoped to its worktree. */}
- {translate('auto.components.sidebar.WorktreeCardPorts.3240f320d7', 'Live Ports')} + {translate('auto.components.sidebar.WorktreeCardPorts.3240f320d7', 'Live Ports')}{' '} + + ({ports.length}) + -
- - - - {ports.length} -
{ports.map((port) => (