scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-17 04:34:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 172

fix(ui): use info tint for clickable table row hover

Browse source 
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
jesus 2026-10-12 22:49:52 +00:00
parent 118d5f566b
commit 91d464e73f
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
ui/litellm-dashboard/src/components/shared/DataTable/DataTable.test.tsx
View file

@ -765,7 +765,7 @@ describe("DataTable pinned columns", () => {
});
describe("DataTable row activation", () => {
it("uses full muted hover tint only for clickable rows", () => {
it("uses info hover tint only for clickable rows", () => {
render(
<>
<DataTable
@ -779,9 +779,9 @@ describe("DataTable row activation", () => {
);
const clickableRow = screen.getByRole("row", { name: "Open person Clickable" });
expect(clickableRow).toHaveClass("hover:bg-muted");
expect(clickableRow).toHaveClass("hover:bg-info/20");
expect(clickableRow).not.toHaveClass("hover:bg-muted/50");
expect(screen.getByRole("row", { name: "Static" })).not.toHaveClass("hover:bg-muted");
expect(screen.getByRole("row", { name: "Static" })).not.toHaveClass("hover:bg-info/20");
});
it("fires onRowClick exactly once from a plain cell but not from interactive elements", async () => {

2
ui/litellm-dashboard/src/components/shared/DataTable/DataTable.tsx
View file

@ -322,7 +322,7 @@ function DataTableBodyRow<TData>({
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 hover:bg-info/20 focus-visible:outline-2 focus-visible:outline-primary focus-visible:-outline-offset-2"
: "",
size === "compact" ? "h-8" : "",
rowClassName?.(row),