diff --git a/e2e-tests/delete_provider.spec.ts b/e2e-tests/delete_provider.spec.ts new file mode 100644 index 0000000..9348056 --- /dev/null +++ b/e2e-tests/delete_provider.spec.ts @@ -0,0 +1,12 @@ +import { test } from "./helpers/test_helper"; + +test("delete custom provider should not freeze", async ({ po }) => { + await po.setUp(); + await po.goToSettingsTab(); + await po.page.getByTestId("custom-provider-more-options").click(); + await po.page.getByRole("button", { name: "Delete Provider" }).click(); + await po.page.getByRole("button", { name: "Delete Provider" }).click(); + + // Make sure UI hasn't freezed + await po.goToAppsTab(); +}); diff --git a/src/components/ProviderSettings.tsx b/src/components/ProviderSettings.tsx index fe80833..5fe55dd 100644 --- a/src/components/ProviderSettings.tsx +++ b/src/components/ProviderSettings.tsx @@ -17,11 +17,11 @@ import { AlertTriangle } from "lucide-react"; import { useState } from "react"; import { - DropdownMenu, - DropdownMenuContent, - DropdownMenuItem, - DropdownMenuTrigger, -} from "@/components/ui/dropdown-menu"; + Popover, + PopoverContent, + PopoverTrigger, +} from "@/components/ui/popover"; +import { Button } from "@/components/ui/button"; import { AlertDialog, AlertDialogAction, @@ -143,22 +143,29 @@ export function ProviderSettingsGrid() { className="absolute top-2 right-2" onClick={(e) => e.stopPropagation()} > - - -
+ + +
-
- - + + + + + )}