Skip to content

Commit d8063d8

Browse files
Copilotsawka
andcommitted
fix: polish vertical tab indicator and close affordance
Co-authored-by: sawka <2722291+sawka@users.noreply.github.com>
1 parent c53ef4a commit d8063d8

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

frontend/app/tab/vtab.tsx

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ export function VTab({
4444
onDrop={onDrop}
4545
onDragEnd={onDragEnd}
4646
className={cn(
47-
"group flex h-9 w-full cursor-pointer items-center gap-2 rounded-md border px-2 text-sm transition-colors select-none",
47+
"group flex h-9 w-full cursor-pointer items-center gap-2 rounded-md border pl-2 pr-1 text-sm transition-colors select-none",
4848
"whitespace-nowrap",
4949
active
5050
? "border-accent/40 bg-accent/20 text-primary"
@@ -53,20 +53,19 @@ export function VTab({
5353
: "border-transparent bg-transparent text-secondary hover:border-border hover:bg-hover",
5454
isDragging && "opacity-50"
5555
)}
56-
title={tab.name}
5756
>
58-
<span className="min-w-0 flex-1 overflow-hidden text-ellipsis whitespace-nowrap">{tab.name}</span>
5957
{tab.indicator && (
6058
<span className="shrink-0 text-xs" style={{ color: tab.indicator.color || "#fbbf24" }}>
6159
<i className={makeIconClass(tab.indicator.icon, true, { defaultIcon: "bell" })} />
6260
</span>
6361
)}
62+
<span className="min-w-0 flex-1 overflow-hidden text-ellipsis whitespace-nowrap">{tab.name}</span>
6463
{onClose && (
6564
<button
6665
type="button"
6766
className={cn(
68-
"shrink-0 cursor-pointer rounded p-1 text-secondary transition",
69-
isReordering ? "opacity-0" : "opacity-0 group-hover:opacity-100 hover:bg-hoverbg hover:text-primary"
67+
"shrink-0 -mr-1 cursor-pointer p-1 text-secondary transition",
68+
isReordering ? "opacity-0" : "opacity-0 group-hover:opacity-100 hover:text-primary"
7069
)}
7170
onClick={(event) => {
7271
event.stopPropagation();

0 commit comments

Comments
 (0)