From ed788c99fa9cab61200ebd23d7f75f39dd2bc6d7 Mon Sep 17 00:00:00 2001 From: jesus Date: Tue, 13 Oct 2026 18:35:23 +0000 Subject: [PATCH] revert: tint clickable table row hover Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --- .../shared/DataTable/DataTable.test.tsx | 19 ------------------- .../components/shared/DataTable/DataTable.tsx | 2 +- 2 files changed, 1 insertion(+), 20 deletions(-) diff --git a/ui/litellm-dashboard/src/components/shared/DataTable/DataTable.test.tsx b/ui/litellm-dashboard/src/components/shared/DataTable/DataTable.test.tsx index 3fdfaf9c994..5cf0e78910e 100644 --- a/ui/litellm-dashboard/src/components/shared/DataTable/DataTable.test.tsx +++ b/ui/litellm-dashboard/src/components/shared/DataTable/DataTable.test.tsx @@ -765,25 +765,6 @@ describe("DataTable pinned columns", () => { }); describe("DataTable row activation", () => { - it("uses full muted hover tint only for clickable rows", () => { - render( - <> - `Open person ${row.name}`} - /> - - , - ); - - const clickableRow = screen.getByRole("row", { name: "Open person Clickable" }); - expect(clickableRow).toHaveClass("hover:bg-muted"); - expect(clickableRow).not.toHaveClass("hover:bg-muted/50"); - expect(screen.getByRole("row", { name: "Static" })).not.toHaveClass("hover:bg-muted"); - }); - it("fires onRowClick exactly once from a plain cell but not from interactive elements", async () => { const user = userEvent.setup(); const onRowClick = vi.fn(); diff --git a/ui/litellm-dashboard/src/components/shared/DataTable/DataTable.tsx b/ui/litellm-dashboard/src/components/shared/DataTable/DataTable.tsx index dcd84188dd8..5219a07f966 100644 --- a/ui/litellm-dashboard/src/components/shared/DataTable/DataTable.tsx +++ b/ui/litellm-dashboard/src/components/shared/DataTable/DataTable.tsx @@ -322,7 +322,7 @@ function DataTableBodyRow({ aria-label={clickable ? getRowLabel?.(row.original) : undefined} className={cn( clickable - ? "cursor-pointer hover:bg-muted focus-visible:outline-2 focus-visible:outline-primary focus-visible:-outline-offset-2" + ? "cursor-pointer focus-visible:outline-2 focus-visible:outline-primary focus-visible:-outline-offset-2" : "", size === "compact" ? "h-8" : "", rowClassName?.(row),