Skip to content

Commit 7ab2fee

Browse files
committed
fix(logs): right-size and reorganize log-detail action chips
Two concrete regressions from the earlier Chip conversion (#5341): - Both chips used variant='primary' (the solid inverse-fill treatment), the heaviest chrome in the design system, reserved for a single standout action per context (Save, Upgrade, Add key) — never a peer among several row actions. Two solid pills stacked in a narrow side panel read as oversized. Switched both to the bare (default) chip, matching every analogous row action in Settings. - The Version badge was size='md' while its siblings (Level, Trigger) are size='sm' — an inconsistency. Aligned to 'sm'. Also folded the floating 'Troubleshoot in Chat' chip into the Details card as its own row (matching 'Snapshot' exactly) instead of leaving it orphaned below Workflow Output — every row in the card now shares one shape (label left, trailing content right), consistent top to bottom.
1 parent e01123f commit 7ab2fee

1 file changed

Lines changed: 14 additions & 20 deletions

File tree

  • apps/sim/app/workspace/[workspaceId]/logs/components/log-details

apps/sim/app/workspace/[workspaceId]/logs/components/log-details/log-details.tsx

Lines changed: 14 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -529,7 +529,7 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
529529
Version
530530
</span>
531531
<div className='flex w-0 flex-1 justify-end'>
532-
<Badge variant='green' size='md' className='max-w-full truncate'>
532+
<Badge variant='green' size='sm' className='max-w-full truncate'>
533533
{log.deploymentVersionName || `v${log.deploymentVersion}`}
534534
</Badge>
535535
</div>
@@ -542,16 +542,23 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
542542
<span className='font-medium text-[var(--text-tertiary)] text-caption'>
543543
Snapshot
544544
</span>
545-
<Chip
546-
variant='primary'
547-
leftIcon={Eye}
548-
flush
549-
onClick={() => setIsExecutionSnapshotOpen(true)}
550-
>
545+
<Chip leftIcon={Eye} flush onClick={() => setIsExecutionSnapshotOpen(true)}>
551546
View Snapshot
552547
</Chip>
553548
</div>
554549
)}
550+
551+
{/* Troubleshoot */}
552+
{canTroubleshoot && (
553+
<div className='flex h-10 items-center justify-between px-3'>
554+
<span className='font-medium text-[var(--text-tertiary)] text-caption'>
555+
Troubleshoot
556+
</span>
557+
<Chip leftIcon={Wrench} flush onClick={handleTroubleshoot}>
558+
Troubleshoot in Chat
559+
</Chip>
560+
</div>
561+
)}
555562
</div>
556563

557564
{/* Workflow Input */}
@@ -581,19 +588,6 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
581588
</div>
582589
)}
583590

584-
{/* Troubleshoot */}
585-
{canTroubleshoot && (
586-
<Chip
587-
variant='primary'
588-
leftIcon={Wrench}
589-
flush
590-
className='self-start'
591-
onClick={handleTroubleshoot}
592-
>
593-
Troubleshoot in Chat
594-
</Chip>
595-
)}
596-
597591
{/* Files */}
598592
{log.files && log.files.length > 0 && <FileCards files={log.files} isExecutionFile />}
599593

0 commit comments

Comments
 (0)