fix(ui): truncate long team names in the models table team dropdown - #34689
Merged
yuneng-berri merged 2 commits intoJul 27, 2026
Merged
Conversation
The Team dropdown popup is pinned to the trigger width via w-(--anchor-width) and clips its overflow, while Base UI's ItemText wrapper is flex-1 shrink-0 with min-width: auto, so it sizes itself to the full nowrap label and simply overflows the popup. Teams without a team_alias render their 36-char id, so those options were sliced mid-character with no ellipsis. Clears min-width: auto off the text wrapper and truncates the label at the call site. The underlying gap is in the shared Select primitive, which any long-labelled select in the dashboard will hit; that is left for a separate change.
…pdown-truncate-2b2a3f
Contributor
Greptile SummaryThis PR fixes long team-option labels in the models table dropdown.
Confidence Score: 5/5The PR appears safe to merge, with the dropdown truncation change isolated to team-option presentation. The changed markup allows the flex text wrapper and label to shrink within the anchored popup, applies overflow truncation, and preserves access to the complete label through the title attribute.
|
| Filename | Overview |
|---|---|
| ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/components/AllModelsTable.tsx | Adds shrinkable, truncated team-option labels with full text available through the title attribute; no actionable defect identified. |
| ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/components/AllModelsTable.test.tsx | Adds focused coverage for the long-label markup, truncation classes, and hover text without weakening existing assertions. |
Reviews (1): Last reviewed commit: "Merge branch 'litellm_internal_staging' ..." | Re-trigger Greptile
yuneng-berri
enabled auto-merge
July 27, 2026 16:46
ryan-crabbe-berri
approved these changes
Jul 27, 2026
yuneng-berri
merged commit Jul 27, 2026
2f2e1e7
into
litellm_internal_staging
73 of 74 checks passed
Merged
5 tasks
stvnksslr
pushed a commit
to stvnksslr/litellm
that referenced
this pull request
Aug 3, 2026
…opdown-truncate-2b2a3f fix(ui): truncate long team names in the models table team dropdown (cherry picked from commit 2f2e1e7)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TLDR
Problem this solves:
How it solves it:
min-width: autooff Base UI's text wrapper so the row can size itRelevant issues
Linear ticket
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
@greptileaito re-request a review after pushing changes)Delays in PR merge?
If you're seeing a delay in your PR being merged, ping the LiteLLM Team on Slack (#pr-review).
Screenshots / Proof of Fix
Captured against a live proxy holding 133 real teams, driven with Playwright: real login, real click on the Team trigger, no DOM manipulation. Viewport 1280x720
Before (commit
215f05588d, tip oflitellm_internal_stagingat the time)After (commit
55ff0e10eb)Layout measured in the same runs, reading
clientWidth/scrollWidthoff each option label:2dfbd3b5-d15c-4e67-9bf8-20dfb57bb4742d57298b-af80-460f-93d2-939cfaaa5fc0120014f3-cffa-431f-a42d-d99f74b288c2Personal(short, control)Labels escaping the 161px popup: 54 of 133 before, 0 of 133 after. Computed style on the label goes from
text-overflow: clip; overflow: visibletotext-overflow: ellipsis; overflow: hiddenTo reproduce by hand: open the dashboard at
/ui/?page=models, All Models tab, and click theTeampill in the table toolbar. Any team without ateam_aliasshows its id, which is what overflowsType
🐛 Bug Fix
Changes
The dropdown popup is pinned to the trigger width by
w-(--anchor-width)and clips its overflow withoverflow-x: hidden. Inside it, Base UI'sItemTextwrapper isflex-1 shrink-0withmin-width: auto, which resolves to the min-content width of nowrap text; a 36-char team id measures 284px and the wrapper sizes itself to that, overflowing the 161px popup and getting clipped at the edge. Nothing in the chain ever gave the text a box smaller than itself, so no ellipsis could appearThe fix clears
min-width: autooff that wrapper from the call site so the row's own width governs it, then truncates the label and keeps the full value in atitleso a truncated id is still recoverable on hoverWorth flagging for the reviewer: this is a call-site fix for a gap in the shared
Selectprimitive.SelectItemrenders its text through a wrapper that can never shrink, so every long-labelled select in the dashboard has the same latent behaviour. Fixing it incomponents/ui/select.tsxwould cover all of them at once, but that file is shadcn-CLI-managed and the change would touch every select in the app, so it belongs in its own PR rather than riding along with this oneFinal Attestation