Skip to content

Commit 769802f

Browse files
committed
More theme fixes
1 parent 3575274 commit 769802f

File tree

6 files changed

+23
-16
lines changed

6 files changed

+23
-16
lines changed

webview-ui/src/components/ui/button.tsx

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -5,22 +5,21 @@ import { cva, type VariantProps } from "class-variance-authority"
55
import { cn } from "@/lib/utils"
66

77
const buttonVariants = cva(
8-
"inline-flex items-center justify-center gap-2 whitespace-nowrap rounded-xs text-base font-medium transition-colors focus-visible:outline-none focus-visible:ring-1 focus-visible:ring-ring disabled:pointer-events-none disabled:opacity-50 [&_svg]:pointer-events-none [&_svg]:size-4 [&_svg]:shrink-0",
8+
"inline-flex items-center justify-center gap-2 whitespace-nowrap rounded-xs text-base font-medium transition-colors focus-visible:outline-none focus-visible:ring-1 focus-visible:ring-ring disabled:pointer-events-none disabled:opacity-50 [&_svg]:pointer-events-none [&_svg]:size-4 [&_svg]:shrink-0 cursor-pointer",
99
{
1010
variants: {
1111
variant: {
1212
default:
13-
"border border-vscode-input-border bg-primary text-primary-foreground shadow hover:bg-primary/90 cursor-pointer",
14-
destructive:
15-
"bg-destructive text-destructive-foreground shadow-sm hover:bg-destructive/90 cursor-pointer",
13+
"border border-vscode-input-border bg-primary text-primary-foreground shadow hover:bg-primary/90",
14+
destructive: "bg-destructive text-destructive-foreground shadow-sm hover:bg-destructive/90",
1615
outline:
17-
"border border-vscode-input-border bg-background shadow-sm hover:bg-accent hover:text-accent-foreground cursor-pointer",
16+
"border border-vscode-input-border bg-background shadow-sm hover:bg-accent hover:text-accent-foreground",
1817
secondary:
19-
"border border-vscode-input-border bg-secondary text-secondary-foreground shadow-sm hover:bg-secondary/80 cursor-pointer",
20-
ghost: "hover:bg-accent hover:text-accent-foreground cursor-pointer",
21-
link: "text-primary underline-offset-4 hover:underline cursor-pointer",
18+
"border border-vscode-input-border bg-secondary text-secondary-foreground shadow-sm hover:bg-secondary/80",
19+
ghost: "hover:bg-accent hover:text-accent-foreground",
20+
link: "text-primary underline-offset-4 hover:underline",
2221
combobox:
23-
"text-vscode-font-size font-normal text-popover-foreground bg-vscode-input-background border border-vscode-dropdown-border hover:bg-vscode-input-background/80 cursor-pointer",
22+
"border border-vscode-input-border bg-vscode-input-background text-muted-foreground shadow-sm hover:bg-transparent",
2423
},
2524
size: {
2625
default: "h-7 px-3",

webview-ui/src/components/ui/command.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,8 @@ const CommandItem = React.forwardRef<
108108
<CommandPrimitive.Item
109109
ref={ref}
110110
className={cn(
111-
"relative flex cursor-pointer gap-2 select-none items-center rounded-sm px-2 py-1.5 text-base text-vscode-dropdown-foreground outline-none data-[disabled=true]:pointer-events-none data-[selected=true]:bg-vscode-list-activeSelectionBackground data-[disabled=true]:opacity-50 [&_svg]:pointer-events-none [&_svg]:size-4 [&_svg]:shrink-0",
111+
"relative flex cursor-pointer gap-2 select-none items-center rounded-sm px-2 py-1.5 text-base text-vscode-dropdown-foreground outline-none data-[disabled=true]:pointer-events-none data-[disabled=true]:opacity-50 [&_svg]:pointer-events-none [&_svg]:size-4 [&_svg]:shrink-0",
112+
"data-[selected=true]:bg-vscode-list-activeSelectionBackground data-[selected=true]:texst-vscode-list-activeSelectionForeground",
112113
className,
113114
)}
114115
{...props}

webview-ui/src/components/ui/dropdown-menu.tsx

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,9 @@ const DropdownMenuSubTrigger = React.forwardRef<
2626
<DropdownMenuPrimitive.SubTrigger
2727
ref={ref}
2828
className={cn(
29-
"flex cursor-default gap-2 select-none items-center rounded-sm px-2 py-1.5 text-sm outline-none focus:bg-vscode-list-activeSelectionBackground data-[state=open]:bg-vscode-list-activeSelectionBackground [&_svg]:pointer-events-none [&_svg]:size-4 [&_svg]:shrink-0",
29+
"flex cursor-default gap-2 select-none items-center rounded-sm px-2 py-1.5 text-sm outline-none [&_svg]:pointer-events-none [&_svg]:size-4 [&_svg]:shrink-0",
30+
"focus:bg-vscode-list-activeSelectionBackground focus:text-vscode-list-activeSelectionForeground",
31+
"data-[state=open]:bg-vscode-list-activeSelectionBackground data-[state=open]:text-vscode-list-activeSelectionForeground",
3032
inset && "pl-8",
3133
className,
3234
)}
@@ -80,7 +82,8 @@ const DropdownMenuItem = React.forwardRef<
8082
<DropdownMenuPrimitive.Item
8183
ref={ref}
8284
className={cn(
83-
"relative flex cursor-pointer select-none items-center gap-2 rounded-sm px-2 py-1.5 text-sm outline-none transition-colors focus:bg-vscode-list-activeSelectionBackground focus:text-accent-foreground data-[disabled]:pointer-events-none data-[disabled]:opacity-50 [&>svg]:size-4 [&>svg]:shrink-0 active:opacity-90",
85+
"relative flex cursor-pointer select-none items-center gap-2 rounded-sm px-2 py-1.5 text-sm outline-none transition-colors data-[disabled]:pointer-events-none data-[disabled]:opacity-50 [&>svg]:size-4 [&>svg]:shrink-0 active:opacity-90",
86+
"focus:bg-vscode-list-activeSelectionBackground focus:text-vscode-list-activeSelectionForeground",
8487
inset && "pl-8",
8588
className,
8689
)}
@@ -96,7 +99,8 @@ const DropdownMenuCheckboxItem = React.forwardRef<
9699
<DropdownMenuPrimitive.CheckboxItem
97100
ref={ref}
98101
className={cn(
99-
"relative flex cursor-default select-none items-center rounded-sm py-1.5 pl-8 pr-2 text-sm outline-none transition-colors focus:bg-vscode-list-activeSelectionBackground focus:text-accent-foreground data-[disabled]:pointer-events-none data-[disabled]:opacity-50",
102+
"relative flex cursor-default select-none items-center rounded-sm py-1.5 pl-8 pr-2 text-sm outline-none transition-colors data-[disabled]:pointer-events-none data-[disabled]:opacity-50",
103+
"focus:bg-vscode-list-activeSelectionBackground focus:text-vscode-list-activeSelectionForeground",
100104
className,
101105
)}
102106
checked={checked}

webview-ui/src/components/ui/select-dropdown.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ export const SelectDropdown = React.forwardRef<React.ElementRef<typeof DropdownM
141141
<DropdownMenuItem
142142
key={`item-${option.value}`}
143143
disabled={option.disabled}
144-
className="text-xs focus:bg-vscode-list-activeSelectionBackground cursor-pointer"
144+
className="text-xs focus:bg-vscode-list-activeSelectionBackground focus:text-vscode-list-activeSelectionForeground cursor-pointer"
145145
onClick={() => handleSelect(option)}>
146146
{option.label}
147147
{option.value === value && (

webview-ui/src/components/ui/select.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@ function SelectTrigger({ className, children, ...props }: React.ComponentProps<t
2222
<SelectPrimitive.Trigger
2323
data-slot="select-trigger"
2424
className={cn(
25-
"border-vscode-input-border data-[placeholder]:text-muted-foreground [&_svg:not([class*='text-'])]:text-muted-foreground focus-visible:border-vscode-focusBorder aria-invalid:border-destructive flex h-7 w-fit items-center justify-between gap-2 rounded-xs border bg-vscode-input-background hover:bg-transparent px-3 py-2 text-sm whitespace-nowrap shadow-xs transition-[color,box-shadow] outline-none disabled:cursor-not-allowed disabled:opacity-50 *:data-[slot=select-value]:line-clamp-1 *:data-[slot=select-value]:flex *:data-[slot=select-value]:items-center *:data-[slot=select-value]:gap-2 [&_svg]:pointer-events-none [&_svg]:shrink-0 [&_svg:not([class*='size-'])]:size-4 cursor-pointer",
25+
"border-vscode-input-border data-[placeholder]:text-muted-foreground [&_svg:not([class*='text-'])]:text-muted-foreground focus-visible:border-vscode-focusBorder aria-invalid:border-destructive flex h-7 w-fit items-center justify-between gap-2 rounded-xs border px-3 py-2 text-sm whitespace-nowrap shadow-xs transition-[color,box-shadow] outline-none disabled:cursor-not-allowed disabled:opacity-50 *:data-[slot=select-value]:line-clamp-1 *:data-[slot=select-value]:flex *:data-[slot=select-value]:items-center *:data-[slot=select-value]:gap-2 [&_svg]:pointer-events-none [&_svg]:shrink-0 [&_svg:not([class*='size-'])]:size-4 cursor-pointer",
26+
"bg-vscode-input-background hover:bg-transparent",
2627
className,
2728
)}
2829
{...props}>
@@ -83,7 +84,8 @@ function SelectItem({ className, children, ...props }: React.ComponentProps<type
8384
<SelectPrimitive.Item
8485
data-slot="select-item"
8586
className={cn(
86-
"focus:bg-vscode-list-activeSelectionBackground focus:text-accent-foreground [&_svg:not([class*='text-'])]:text-muted-foreground relative flex w-full items-center gap-2 rounded-xs py-1.5 pr-8 pl-2 text-sm outline-hidden select-none data-[disabled]:pointer-events-none data-[disabled]:opacity-50 [&_svg]:pointer-events-none [&_svg]:shrink-0 [&_svg:not([class*='size-'])]:size-4 *:[span]:last:flex *:[span]:last:items-center *:[span]:last:gap-2 cursor-pointer",
87+
"[&_svg:not([class*='text-'])]:text-muted-foreground relative flex w-full items-center gap-2 rounded-xs py-1.5 pr-8 pl-2 text-sm outline-hidden select-none data-[disabled]:pointer-events-none data-[disabled]:opacity-50 [&_svg]:pointer-events-none [&_svg]:shrink-0 [&_svg:not([class*='size-'])]:size-4 *:[span]:last:flex *:[span]:last:items-center *:[span]:last:gap-2 cursor-pointer",
88+
"focus:bg-vscode-list-activeSelectionBackground focus:text-vscode-list-activeSelectionForeground",
8789
className,
8890
)}
8991
{...props}>

webview-ui/src/index.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@
9797
--color-vscode-list-hoverBackground: var(--vscode-list-hoverBackground);
9898
--color-vscode-list-focusBackground: var(--vscode-list-focusBackground);
9999
--color-vscode-list-activeSelectionBackground: var(--vscode-list-activeSelectionBackground);
100+
--color-vscode-list-activeSelectionForeground: var(--vscode-list-activeSelectionForeground);
100101

101102
--color-vscode-toolbar-hoverBackground: var(--vscode-toolbar-hoverBackground);
102103

0 commit comments

Comments
 (0)