Skip to content

Studio: Fix clipped model selector text descenders - #5210

Merged
danielhanchen merged 7 commits into
unslothai:mainfrom
Imagineer99:fix/model-selector-descender-clipping
Apr 29, 2026
Merged

Studio: Fix clipped model selector text descenders#5210
danielhanchen merged 7 commits into
unslothai:mainfrom
Imagineer99:fix/model-selector-descender-clipping

Conversation

@Imagineer99

Copy link
Copy Markdown
Collaborator

Summary

Fixes a visual issue where model names containing descenders, such as g, p, or q, could appear clipped at the bottom in the model selector trigger.

The label previously used leading-none together with truncate, which applies overflow: hidden. That made the text box tight enough for some font descenders to be clipped. This changes the label to leading-tight, keeping truncation behavior while giving the glyphs enough vertical room.

Before:
image

After:
image

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request updates the styling of the model selector trigger in the assistant UI, specifically changing the line height of the model name from leading-none to leading-tight. I have no feedback to provide.

@Imagineer99
Imagineer99 marked this pull request as ready for review April 28, 2026 14:58
* fix: allow image-only chat messages

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* test: deduplicate empty content validation coverage

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@danielhanchen danielhanchen added the auto-reviewing Auto-review in progress label Apr 29, 2026
Replace `leading-none` with `leading-tight` on the parent div wrapping
`displayTitle` and the "Studio" label inside `SidebarMenuButton`. The
child spans use `truncate` (overflow: hidden), so `line-height: 1`
clipped descenders (g, p, q, y, j) on user names. Same root cause and
fix as the model selector trigger.
@danielhanchen
danielhanchen self-requested a review as a code owner April 29, 2026 03:21
@danielhanchen danielhanchen added auto-approved Auto-review approved the PR and removed auto-reviewing Auto-review in progress labels Apr 29, 2026
@danielhanchen

Copy link
Copy Markdown
Member

Auto-review verdict: Approved

PR #5210 swaps Tailwind leading-none to leading-tight on the model-selector trigger label so glyph descenders (g/p/q/y) stop being clipped by truncate's overflow:hidden. Review extended the same one-class fix to the sidebar user-account block where the identical pattern caused identical clipping for user names.

Reason: Real descender-clipping bug fixed at the original location and at one parallel location found during review; all reviewers across 3 iterations approve, no remaining bugs.

@danielhanchen
danielhanchen merged commit 146295e into unslothai:main Apr 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-approved Auto-review approved the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants