Skip to content

Commit 32ce9db

Browse files
ahalekellyclaude
andauthored
fix(web): remote machine icon no longer forces a third sidebar line (#26803)
Co-authored-by: Claude Opus 5.5 <[email protected]>
1 parent 35231d7 commit 32ce9db

1 file changed

Lines changed: 13 additions & 15 deletions

File tree

‎apps/web/src/components/sidebar/SidebarThreadRow.tsx‎

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1122,14 +1122,19 @@ const SidebarThreadRow = memo(function SidebarThreadRow(props: SidebarThreadRowP
11221122
)
11231123
) : null;
11241124

1125+
const remoteMachineIcon =
1126+
isRemote && !showsScratchMachine ? (
1127+
<span className="inline-flex shrink-0 items-center text-sidebar-muted-foreground/70">
1128+
<EnvironmentMachineIcon aria-hidden kind={props.environmentMachine} className="size-3.5" />
1129+
</span>
1130+
) : null;
11251131
const diff = latestRunDiff(thread);
1126-
// Only the provider icon would remain, so the row drops the line instead of leaving it blank.
1132+
// Only the trailing icons would remain, so the row moves them to the title line instead.
11271133
const metaLineEmpty =
11281134
!thread.branch &&
11291135
!showsScratchMachine &&
11301136
prBadge === null &&
11311137
diff === null &&
1132-
!isRemote &&
11331138
runningTerminalIds.length === 0;
11341139
const metaLine = metaLineEmpty ? null : (
11351140
<div className="mt-0.5 flex min-w-0 items-center gap-1.5 text-secondary-label text-xs">
@@ -1164,24 +1169,17 @@ const SidebarThreadRow = memo(function SidebarThreadRow(props: SidebarThreadRowP
11641169
aria-hidden
11651170
className="pointer-events-none ml-auto inline-flex shrink-0 items-center gap-1"
11661171
>
1167-
{isRemote && !showsScratchMachine ? (
1168-
<span className="inline-flex shrink-0 items-center text-sidebar-muted-foreground/70">
1169-
<EnvironmentMachineIcon
1170-
aria-hidden
1171-
kind={props.environmentMachine}
1172-
className="size-3.5"
1173-
/>
1174-
</span>
1175-
) : null}
1172+
{remoteMachineIcon}
11761173
<SidebarProviderStack
11771174
thread={thread}
11781175
providerEntryByInstanceId={props.providerEntryByInstanceId}
11791176
/>
11801177
</span>
11811178
</div>
11821179
);
1183-
const titleLineProviderStack = metaLineEmpty ? (
1184-
<span aria-hidden className="inline-flex shrink-0">
1180+
const titleLineIcons = metaLineEmpty ? (
1181+
<span aria-hidden className="inline-flex shrink-0 items-center gap-1">
1182+
{remoteMachineIcon}
11851183
<SidebarProviderStack
11861184
thread={thread}
11871185
providerEntryByInstanceId={props.providerEntryByInstanceId}
@@ -1249,7 +1247,7 @@ const SidebarThreadRow = memo(function SidebarThreadRow(props: SidebarThreadRowP
12491247
)}
12501248
{draftIndicator}
12511249
{title}
1252-
{multiline ? titleLineProviderStack : null}
1250+
{multiline ? titleLineIcons : null}
12531251
{pinIndicator}
12541252
{multiline ? null : terminalStatusIcon}
12551253
{isRegeneratingTitle ? (
@@ -1569,7 +1567,7 @@ const SidebarThreadRow = memo(function SidebarThreadRow(props: SidebarThreadRowP
15691567
Regenerating title
15701568
</span>
15711569
) : null}
1572-
{titleLineProviderStack}
1570+
{titleLineIcons}
15731571
</div>
15741572
{metaLine}
15751573
</div>

0 commit comments

Comments
 (0)