Skip to content

fix(ui): give model leaderboard a distinct trophy icon - #44036

Merged
moe-berri merged 1 commit into
mainfrom
litellm_leaderboard_sidebar_icon
Oct 1, 2026
Merged

moe-berri merged 1 commit into
mainfrom
litellm_leaderboard_sidebar_icon

Conversation

@moe-berri

@moe-berri moe-berri commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

TLDR

Problem this solves:

  • Model Leaderboard duplicates the Usage sidebar icon

How it solves it:

  • Use the existing Lucide trophy icon for Model Leaderboard

Intentional product change: Model Leaderboard has a distinct trophy icon; navigation and permissions stay the same

User Flow

Before: Usage and Model Leaderboard share a bar-chart icon

  1. Open /ui/model-insights/
  2. See identical icons beside Usage and Model Leaderboard

After: the leaderboard is recognizable by its trophy icon

  1. Open /ui/model-insights/
  2. See a trophy beside Model Leaderboard and a chart beside Usage

Validation

All 43 existing sidebar tests pass; focused ESLint has zero errors and five existing warnings. No tests were modified or added for this visual-only change

Caveats

Low

  • Browser screenshot verification and CI are pending; kept as draft

Note

Low Risk
Visual-only sidebar icon change in leftnav.tsx with no routing, auth, or API impact.

Overview
Model Leaderboard in the left sidebar no longer shares the bar-chart icon with Usage. The model-insights nav item now uses Lucide’s Trophy icon instead of BarChart3, with a matching import in leftnav.tsx.

Routes, labels, roles, and permissions are unchanged—this is purely visual differentiation in the Observability section.

Reviewed by Cursor Bugbot for commit dc27a83. Bugbot is set up for automated code reviews on this repo. Configure here.

@moe-berri

Copy link
Copy Markdown
Contributor Author

bugbot run

@moe-berri

Copy link
Copy Markdown
Contributor Author

@greptile-ai

@moe-berri

Copy link
Copy Markdown
Contributor Author

@veria-ai

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit dc27a83. Configure here.

@moe-berri
moe-berri marked this pull request as ready for review October 1, 2026 17:20
@moe-berri
moe-berri requested a review from a team October 1, 2026 17:20
@moe-berri
moe-berri merged commit 91ff045 into main Oct 1, 2026
96 of 98 checks passed
@moe-berri
moe-berri deleted the litellm_leaderboard_sidebar_icon branch October 1, 2026 17:20
@greptile-apps

greptile-apps Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Low risk] Changes a navigation icon in the dashboard UI.

The PR appears safe to merge; no actionable issue was identified.

Summary

The Model Leaderboard sidebar item now uses a trophy icon instead of the chart icon shared with Usage. Its route, label, and permissions are unchanged.

Reviews (1) · Last reviewed commit: "fix(ui): give model leaderboard a distin..."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants