Skip to content

Commit 17cd758

Browse files
committed
chore: update to use DropdownMenu
1 parent a8fe0bf commit 17cd758

File tree

1 file changed

+27
-17
lines changed

1 file changed

+27
-17
lines changed

site/src/pages/DeploymentSettingsPage/AppearanceSettingsPage/AnnouncementBannerItem.tsx

Lines changed: 27 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,9 @@ import Checkbox from "@mui/material/Checkbox";
33
import TableCell from "@mui/material/TableCell";
44
import TableRow from "@mui/material/TableRow";
55
import type { BannerConfig } from "api/typesGenerated";
6-
import {
7-
MoreMenu,
8-
MoreMenuContent,
9-
MoreMenuItem,
10-
MoreMenuTrigger,
11-
ThreeDotsButton,
12-
} from "components/MoreMenu/MoreMenu";
6+
import { Button } from "components/Button/Button";
7+
import { DropdownMenu, DropdownMenuContent, DropdownMenuItem, DropdownMenuTrigger } from "components/DropdownMenu/DropdownMenu";
8+
import { EllipsisVertical } from "lucide-react";
139
import type { FC } from "react";
1410

1511
interface AnnouncementBannerItemProps {
@@ -48,17 +44,31 @@ export const AnnouncementBannerItem: FC<AnnouncementBannerItemProps> = ({
4844
</TableCell>
4945

5046
<TableCell>
51-
<MoreMenu>
52-
<MoreMenuTrigger>
53-
<ThreeDotsButton />
54-
</MoreMenuTrigger>
55-
<MoreMenuContent>
56-
<MoreMenuItem onClick={() => onEdit()}>Edit&hellip;</MoreMenuItem>
57-
<MoreMenuItem onClick={() => onDelete()} danger>
47+
<DropdownMenu>
48+
<DropdownMenuTrigger asChild>
49+
<Button
50+
size="icon-lg"
51+
variant="subtle"
52+
aria-label="Open menu"
53+
>
54+
<EllipsisVertical aria-hidden="true" />
55+
<span className="sr-only">Open menu</span>
56+
</Button>
57+
</DropdownMenuTrigger>
58+
<DropdownMenuContent align="end">
59+
<DropdownMenuItem
60+
onClick={() => onEdit()}
61+
>
62+
Edit&hellip;
63+
</DropdownMenuItem>
64+
<DropdownMenuItem
65+
className="text-content-destructive focus:text-content-destructive"
66+
onClick={() => onDelete()}
67+
>
5868
Delete&hellip;
59-
</MoreMenuItem>
60-
</MoreMenuContent>
61-
</MoreMenu>
69+
</DropdownMenuItem>
70+
</DropdownMenuContent>
71+
</DropdownMenu>
6272
</TableCell>
6373
</TableRow>
6474
);

0 commit comments

Comments
 (0)