Skip to content

Commit 398d954

Browse files
committed
fix(logs): hover only the interactive Run ID row in log details
The detail-card rows all hovered to --surface-2, but the card itself is --surface-2, so the hover was a no-op in light mode and only showed in dark. It also implied clickability on static readout rows. Now only the clickable Run ID row hovers, using the canonical --surface-active token; static rows carry no hover.
1 parent 42c53d0 commit 398d954

1 file changed

Lines changed: 9 additions & 12 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: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -474,7 +474,7 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
474474
role='button'
475475
tabIndex={0}
476476
aria-label='Copy run ID'
477-
className='flex h-10 min-w-0 cursor-pointer items-center justify-between gap-4 px-3 transition-colors hover-hover:bg-[var(--surface-2)]'
477+
className='flex h-10 min-w-0 cursor-pointer items-center justify-between gap-4 px-3 transition-colors hover-hover:bg-[var(--surface-active)]'
478478
onClick={() => copyRunId(log.executionId!)}
479479
onKeyDown={(event) =>
480480
handleKeyboardActivation(event, () => copyRunId(log.executionId!))
@@ -490,15 +490,15 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
490490
)}
491491

492492
{/* Level */}
493-
<div className='flex h-10 items-center justify-between px-3 transition-colors hover-hover:bg-[var(--surface-2)]'>
493+
<div className='flex h-10 items-center justify-between px-3'>
494494
<span className='font-medium text-[var(--text-tertiary)] text-caption'>
495495
Level
496496
</span>
497497
<StatusBadge status={logStatus} />
498498
</div>
499499

500500
{/* Trigger */}
501-
<div className='flex h-10 items-center justify-between px-3 transition-colors hover-hover:bg-[var(--surface-2)]'>
501+
<div className='flex h-10 items-center justify-between px-3'>
502502
<span className='font-medium text-[var(--text-tertiary)] text-caption'>
503503
Trigger
504504
</span>
@@ -512,7 +512,7 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
512512
</div>
513513

514514
{/* Duration */}
515-
<div className='flex h-10 items-center justify-between px-3 transition-colors hover-hover:bg-[var(--surface-2)]'>
515+
<div className='flex h-10 items-center justify-between px-3'>
516516
<span className='font-medium text-[var(--text-tertiary)] text-caption'>
517517
Duration
518518
</span>
@@ -523,7 +523,7 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
523523

524524
{/* Version */}
525525
{log.deploymentVersion && (
526-
<div className='flex h-10 items-center gap-2 px-3 transition-colors hover-hover:bg-[var(--surface-2)]'>
526+
<div className='flex h-10 items-center gap-2 px-3'>
527527
<span className='flex-shrink-0 font-medium text-[var(--text-tertiary)] text-caption'>
528528
Version
529529
</span>
@@ -537,7 +537,7 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
537537

538538
{/* Snapshot */}
539539
{showWorkflowState && (
540-
<div className='flex h-10 items-center justify-between px-3 transition-colors hover-hover:bg-[var(--surface-2)]'>
540+
<div className='flex h-10 items-center justify-between px-3'>
541541
<span className='font-medium text-[var(--text-tertiary)] text-caption'>
542542
Snapshot
543543
</span>
@@ -600,10 +600,7 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
600600
{hasCostInfo && costBreakdown && (
601601
<div className='divide-y divide-[var(--border)] overflow-hidden rounded-md border border-[var(--border)] bg-[var(--surface-2)] dark:bg-transparent'>
602602
{costBreakdown.rows.map((row) => (
603-
<div
604-
key={row.key}
605-
className='flex h-10 items-center justify-between px-3 transition-colors hover-hover:bg-[var(--surface-2)]'
606-
>
603+
<div key={row.key} className='flex h-10 items-center justify-between px-3'>
607604
<span className='min-w-0 truncate font-medium text-[var(--text-tertiary)] text-caption'>
608605
{row.label}
609606
</span>
@@ -612,7 +609,7 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
612609
</span>
613610
</div>
614611
))}
615-
<div className='flex h-10 items-center justify-between px-3 transition-colors hover-hover:bg-[var(--surface-2)]'>
612+
<div className='flex h-10 items-center justify-between px-3'>
616613
<span className='font-medium text-[var(--text-secondary)] text-caption'>
617614
Total
618615
</span>
@@ -621,7 +618,7 @@ export function LogDetailsContent({ log, onActiveTabChange }: LogDetailsContentP
621618
</span>
622619
</div>
623620
{(costBreakdown.tokens.input > 0 || costBreakdown.tokens.output > 0) && (
624-
<div className='flex h-10 items-center justify-between px-3 transition-colors hover-hover:bg-[var(--surface-2)]'>
621+
<div className='flex h-10 items-center justify-between px-3'>
625622
<span className='font-medium text-[var(--text-tertiary)] text-caption'>
626623
Tokens
627624
</span>

0 commit comments

Comments
 (0)