chore: update MoreMenu to use DropdownMenu

jaaydenh · jaaydenh · commit fe820b37f5a1 · 2025-05-01T15:16:31.000Z
diff --git a/site/src/pages/DeploymentSettingsPage/AppearanceSettingsPage/AnnouncementBannerItem.tsx b/site/src/pages/DeploymentSettingsPage/AppearanceSettingsPage/AnnouncementBannerItem.tsx
@@ -4,7 +4,12 @@ import TableCell from "@mui/material/TableCell";
 import TableRow from "@mui/material/TableRow";
 import type { BannerConfig } from "api/typesGenerated";
 import { Button } from "components/Button/Button";
-import { DropdownMenu, DropdownMenuContent, DropdownMenuItem, DropdownMenuTrigger } from "components/DropdownMenu/DropdownMenu";
+import {
+	DropdownMenu,
+	DropdownMenuContent,
+	DropdownMenuItem,
+	DropdownMenuTrigger,
+} from "components/DropdownMenu/DropdownMenu";
 import { EllipsisVertical } from "lucide-react";
 import type { FC } from "react";
 
@@ -46,19 +51,13 @@ export const AnnouncementBannerItem: FC<AnnouncementBannerItemProps> = ({
 			<TableCell>
 				<DropdownMenu>
 					<DropdownMenuTrigger asChild>
-						<Button
-							size="icon-lg"
-							variant="subtle"
-							aria-label="Open menu"
-						>
+						<Button size="icon-lg" variant="subtle" aria-label="Open menu">
 							<EllipsisVertical aria-hidden="true" />
 							<span className="sr-only">Open menu</span>
 						</Button>
 					</DropdownMenuTrigger>
 					<DropdownMenuContent align="end">
-						<DropdownMenuItem
-							onClick={() => onEdit()}
-						>
+						<DropdownMenuItem onClick={() => onEdit()}>
 							Edit&hellip;
 						</DropdownMenuItem>
 						<DropdownMenuItem
diff --git a/site/src/pages/UserSettingsPage/ExternalAuthPage/ExternalAuthPageView.tsx b/site/src/pages/UserSettingsPage/ExternalAuthPage/ExternalAuthPageView.tsx
@@ -18,16 +18,17 @@ import type {
 } from "api/typesGenerated";
 import { ErrorAlert } from "components/Alert/ErrorAlert";
 import { Avatar } from "components/Avatar/Avatar";
-import { Loader } from "components/Loader/Loader";
+import { Button } from "components/Button/Button";
 import {
-	MoreMenu,
-	MoreMenuContent,
-	MoreMenuItem,
-	MoreMenuTrigger,
-	ThreeDotsButton,
-} from "components/MoreMenu/MoreMenu";
+	DropdownMenu,
+	DropdownMenuContent,
+	DropdownMenuItem,
+	DropdownMenuTrigger,
+} from "components/DropdownMenu/DropdownMenu";
+import { Loader } from "components/Loader/Loader";
 import { Stack } from "components/Stack/Stack";
 import { TableEmpty } from "components/TableEmpty/TableEmpty";
+import { EllipsisVertical } from "lucide-react";
 import type { ExternalAuthPollingState } from "pages/CreateWorkspacePage/CreateWorkspacePage";
 import { type FC, useCallback, useEffect, useState } from "react";
 import { useQuery } from "react-query";
@@ -178,12 +179,15 @@ const ExternalAuthRow: FC<ExternalAuthRowProps> = ({
 				</LoadingButton>
 			</TableCell>
 			<TableCell>
-				<MoreMenu>
-					<MoreMenuTrigger>
-						<ThreeDotsButton size="small" disabled={!authenticated} />
-					</MoreMenuTrigger>
-					<MoreMenuContent>
-						<MoreMenuItem
+				<DropdownMenu>
+					<DropdownMenuTrigger asChild>
+						<Button size="icon-lg" variant="subtle" aria-label="Open menu">
+							<EllipsisVertical aria-hidden="true" />
+							<span className="sr-only">Open menu</span>
+						</Button>
+					</DropdownMenuTrigger>
+					<DropdownMenuContent align="end">
+						<DropdownMenuItem
 							onClick={async () => {
 								onValidateExternalAuth();
 								// This is kinda jank. It does a refetch of the thing
@@ -194,19 +198,18 @@ const ExternalAuthRow: FC<ExternalAuthRowProps> = ({
 							}}
 						>
 							Test Validate&hellip;
-						</MoreMenuItem>
-						<Divider />
-						<MoreMenuItem
-							danger
+						</DropdownMenuItem>
+						<DropdownMenuItem
+							className="text-content-destructive focus:text-content-destructive"
 							onClick={async () => {
 								onUnlinkExternalAuth();
 								await refetch();
 							}}
 						>
 							Unlink&hellip;
-						</MoreMenuItem>
-					</MoreMenuContent>
-				</MoreMenu>
+						</DropdownMenuItem>
+					</DropdownMenuContent>
+				</DropdownMenu>
 			</TableCell>
 		</TableRow>
 	);
