Skip to content

Commit c62994e

Browse files
committed
remove unnecessary DocsPopover and more actions changes
1 parent 265adca commit c62994e

File tree

2 files changed

+13
-13
lines changed

2 files changed

+13
-13
lines changed

app/components/DocsPopover.tsx

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,11 @@ export const DocsPopover = ({ heading, icon, summary, links }: DocsPopoverProps)
4646
const title = `Learn about ${heading}`
4747
return (
4848
<Popover>
49-
<PopoverButton title={title}>
50-
<div className={cn(buttonStyle({ size: 'sm', variant: 'ghost' }), 'w-8')}>
51-
<Info16Icon aria-hidden className="shrink-0" />
52-
</div>
49+
<PopoverButton
50+
className={cn(buttonStyle({ size: 'sm', variant: 'ghost' }), 'w-8')}
51+
title={title}
52+
>
53+
<Info16Icon aria-hidden className="shrink-0" />
5354
</PopoverButton>
5455
<PopoverPanel
5556
// popover-panel needed for enter animation

app/components/MoreActionsMenu.tsx

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -27,15 +27,14 @@ export const MoreActionsMenu = ({
2727
}: MoreActionsMenuProps) => {
2828
return (
2929
<DropdownMenu.Root>
30-
<DropdownMenu.Trigger aria-label={label}>
31-
<div
32-
className={cn(
33-
'active-clicked flex items-center justify-center rounded border border-default hover:bg-tertiary',
34-
isSmall ? 'h-6 w-6' : 'h-8 w-8'
35-
)}
36-
>
37-
<More12Icon />
38-
</div>
30+
<DropdownMenu.Trigger
31+
aria-label={label}
32+
className={cn(
33+
'active-clicked flex items-center justify-center rounded border border-default hover:bg-tertiary',
34+
isSmall ? 'h-6 w-6' : 'h-8 w-8'
35+
)}
36+
>
37+
<More12Icon aria-hidden />
3938
</DropdownMenu.Trigger>
4039
<DropdownMenu.Content className="mt-2">{children}</DropdownMenu.Content>
4140
</DropdownMenu.Root>

0 commit comments

Comments
 (0)