scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-10 03:28:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 85

refactor(ui): drop whole-row navigation on the skills table

Browse source 
Only the name cell and the overflow menu act on a row, matching the unified
table pattern; the previous table navigated on any row click
This commit is contained in:
Yuneng Jiang 2026-07-14 19:52:43 -07:00
parent 56655218d0
commit e6c776d508
No known key found for this signature in database
2 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.test.tsx
View file

@ -69,11 +69,11 @@ describe("PluginTable", () => {
expect(mockOnPluginClick).toHaveBeenCalledWith("plugin-id-newer");
});
it("should call onPluginClick with the plugin ID when the row is clicked", async () => {
it("should not navigate when clicking elsewhere in the row", async () => {
const user = userEvent.setup();
render(<PluginTable {...defaultProps} />);
await user.click(screen.getByText("A skill for testing"));
expect(mockOnPluginClick).toHaveBeenCalledWith("plugin-id-newer");
expect(mockOnPluginClick).not.toHaveBeenCalled();
});
it("should badge the category and fall back to Uncategorized", () => {

1
ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.tsx
View file

@ -48,7 +48,6 @@ const PluginTable: React.FC<PluginTableProps> = ({ pluginsList, isLoading, onDel
sortingMode="client"
sorting={sorting}
onSortingChange={setSorting}
onRowClick={(plugin) => onPluginClick(plugin.id)}
isLoading={isLoading}
loadingMessage="Loading skills…"
noDataMessage={<EmptyState />}