Skip to content

Commit

Permalink
fix(GIST-98): update group css class on buttons (#39)
Browse files Browse the repository at this point in the history
  • Loading branch information
dorian-grst authored Nov 13, 2024
1 parent 782ace6 commit 8e057a9
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 14 deletions.
22 changes: 11 additions & 11 deletions src/components/shadcn/button.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,17 +9,17 @@ const buttonVariants = cva(
{
variants: {
variant: {
default: "bg-primary text-primary-foreground hover:bg-primary/90 tranition-all",
destructive: "bg-destructive text-destructive-foreground hover:bg-destructive/90 tranition-all",
outline: "border border-input bg-background hover:bg-accent hover:text-accent-foreground tranition-all",
secondary: "bg-secondary text-secondary-foreground hover:bg-secondary/80 tranition-all",
ghost: "hover:bg-light-background hover:text-foreground tranition-all",
disabled: "hover:bg-light-background hover:text-foreground opacity-50 cursor-not-allowed tranition-all",
link: "text-primary text-base underline-offset-4 hover:underline tranition-all",
icon: "text-foreground bg-icon hover:bg-icon/80 tranition-all",
"icon-ghost": "hover:bg-icon hover:text-foreground tranition-all",
menu: "text-primary-foreground hover:bg-primary hover:text-primary-foreground tranition-all",
header: "hover:bg-primary hover:text-foreground tranition-all",
default: "bg-primary text-primary-foreground hover:bg-primary/90 transition-all",
destructive: "bg-destructive text-destructive-foreground hover:bg-destructive/90 transition-all",
outline: "border border-input bg-background hover:bg-accent hover:text-accent-foreground transition-all",
secondary: "bg-secondary text-secondary-foreground hover:bg-secondary/80 transition-all",
ghost: "hover:bg-light-background hover:text-foreground transition-all",
disabled: "hover:bg-light-background hover:text-foreground opacity-50 cursor-not-allowed transition-all",
link: "text-primary text-base underline-offset-4 hover:underline transition-all",
icon: "text-foreground bg-icon hover:bg-icon/80 transition-all",
"icon-ghost": "hover:bg-icon hover:text-foreground transition-all",
menu: "text-primary-foreground hover:bg-primary hover:text-primary-foreground transition-all",
header: "hover:bg-primary hover:text-foreground transition-all",
},
size: {
default: "h-10 px-6 py-3",
Expand Down
4 changes: 2 additions & 2 deletions src/components/ui/menu-button.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,13 @@ const MenuButton = React.forwardRef<HTMLButtonElement, MenuButtonProps>(
ref={ref}
variant={variant}
size={size}
className={cn("flex !gap-2 flex-row justify-between items-center group", className)}
className={cn("flex !gap-2 flex-row justify-between items-center group/button", className)}
{...props}
>
<div className="flex flex-row gap-2 items-center">
{icon &&
React.cloneElement(icon, {
className: "h-4 w-4 text-slate-500 group-hover:text-primary-foreground tranition-all",
className: "h-4 w-4 text-slate-500 group-hover/button:text-primary-foreground transition-all",
})}
<span className="font-normal text-foreground">{children}</span>
</div>
Expand Down
2 changes: 1 addition & 1 deletion src/components/ui/shortcut.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export default function Shortcut({ letter, className }: ShortcutProps) {
return (
<div
className={cn(
"text-[8px] text-foreground border border-slate-500 bg-slate-600 h-[14px] p-1 flex justify-center items-center rounded group-hover:bg-slate-200 group-hover:border-white group-hover:text-background",
"text-[8px] text-foreground border border-slate-500 bg-slate-600 h-[14px] p-1 flex justify-center items-center rounded group-hover/button:bg-slate-200 group-hover/button:border-white group-hover/button:text-background",
className
)}
>
Expand Down

0 comments on commit 8e057a9

Please sign in to comment.