Skip to content

Commit c6eefee

Browse files
chore: address review comments
1 parent ccbc8d8 commit c6eefee

File tree

2 files changed

+12
-3
lines changed

2 files changed

+12
-3
lines changed

apps/sim/components/emcn/components/button/button.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ const buttonVariants = cva(
2424
ghost: 'text-[var(--text-secondary)] hover-hover:text-[var(--text-primary)]',
2525
subtle:
2626
'text-[var(--text-body)] hover-hover:text-[var(--text-body)] hover-hover:bg-[var(--surface-4)]',
27-
'ghost-secondary': 'text-[var(--text-muted)]',
27+
'ghost-secondary': 'text-[var(--text-muted)] hover-hover:text-[var(--text-primary)]',
2828
/** Branded button - requires branded-button-gradient or branded-button-custom class for colors */
2929
branded:
3030
'rounded-[10px] border text-white hover-hover:text-white text-base transition-[transform,background-color,color,border-color] duration-200',

apps/sim/components/emcn/components/popover/popover.tsx

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -616,7 +616,16 @@ const PopoverContent = React.forwardRef<
616616
...style,
617617
}}
618618
>
619-
{children}
619+
{showArrow ? (
620+
<div
621+
className='overflow-auto flex-1 flex flex-col'
622+
style={{ maxHeight: `${maxHeight || 400}px` }}
623+
>
624+
{children}
625+
</div>
626+
) : (
627+
children
628+
)}
620629
{showArrow && (
621630
<PopoverPrimitive.Arrow width={14} height={7} asChild>
622631
<svg
@@ -1189,4 +1198,4 @@ export {
11891198
usePopoverContext,
11901199
}
11911200

1192-
export type { PopoverSize, PopoverColorScheme }
1201+
export type { PopoverSize, PopoverColorScheme }

0 commit comments

Comments
 (0)