Skip to content

Commit 0a4ac3c

Browse files
committed
improvement(chat): drop the icon badge's own hover step
The chip's hover is the only hover affordance needed. The badge fill is now constant, sitting one step below --surface-6 in light mode so the chip's hover fill cannot close on it — which is what the per-badge step was compensating for.
1 parent 159a706 commit 0a4ac3c

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

  • apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/attached-files-list

apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/attached-files-list/attached-files-list.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,10 @@ const AttachedFileChip = React.memo(function AttachedFileChip({
8585
</>
8686
) : (
8787
<>
88-
{/* Hover steps too: --surface-active is within 7/255 of --surface-6 in light mode. */}
89-
<span className='flex size-[32px] shrink-0 items-center justify-center rounded-md bg-[var(--surface-6)] text-[var(--text-icon)] transition-colors hover-hover:group-hover:bg-[var(--surface-7)] dark:bg-[var(--surface-3)]'>
88+
{/* Fill is constant — the chip's own hover is the only hover affordance. It
89+
sits a step below `--surface-6` in light mode so the chip's hover fill
90+
(`--surface-active`) cannot close on it. */}
91+
<span className='flex size-[32px] shrink-0 items-center justify-center rounded-md bg-[var(--surface-7)] text-[var(--text-icon)] dark:bg-[var(--surface-3)]'>
9092
<Icon className='size-[16px]' />
9193
</span>
9294
<span className='flex min-w-0 flex-col items-start'>

0 commit comments

Comments
 (0)