Skip to content

fix(web): unify composer control rounding - #5957

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
kototok903:fix/web-composer-control-radius
Aug 20, 2026
Merged

fix(web): unify composer control rounding#5957
maria-rcks merged 1 commit into
pingdotgg:mainfrom
kototok903:fix/web-composer-control-radius

Conversation

@kototok903

@kototok903 kototok903 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

What Changed

Added rounded-[var(--control-radius)] to shared composer control class (composerControlClassName) to unify rounding on button-backed and select-backed composer controls.

Why

The button-backed Model and Traits pickers inherited the shared 8px control radius (from rounded-[var(--control-radius)] in Button), but the select-backed Runtime mode picker inherited the generic 10px radius (from rounded-lg in Select). This causes Runtime mode control look inconsistent with the rest of the UI.

Defining the explicit radius in the shared class unifies it for all composer controls.

UI Changes

Before

before screenshot

After

after screenshot

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes (N/A)

Note

Low Risk
Cosmetic Tailwind class change on chat composer controls only; no logic or data impact.

Overview
Adds rounded-[var(--control-radius)] to the shared composerControlClassName used by ComposerControl and ComposerSelectControl, so button-backed and select-backed composer controls (e.g. Runtime mode vs Model/Traits) share the same corner radius instead of mixing Button’s control token with SelectTrigger’s rounded-lg.

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

Note

Apply --control-radius border radius to composer controls

Adds rounded-[var(--control-radius)] to composerControlClassName in ComposerControl.tsx so all composer controls use a consistent, theme-driven border radius.

Macroscope summarized 425c5ab.

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 65fc8d78-ed72-4e6d-a64c-46da3e5352d9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 10, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 425c5ab

This is a purely cosmetic CSS change that adds consistent border-radius styling to the composer control using an existing CSS variable. No runtime behavior is affected.

You can customize Macroscope's approvability policy. Learn more.

@kototok903 kototok903 changed the title fix(web): unify composer control radii fix(web): unify composer control rounding Aug 11, 2026
@maria-rcks
maria-rcks enabled auto-merge (squash) August 20, 2026 20:39
@maria-rcks
maria-rcks merged commit cd14b3e into pingdotgg:main Aug 20, 2026
17 checks passed
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 20, 2026
## What's Changed
* fix(web): retry failed thread bootstraps with a fresh id by @SunkenInTime in pingdotgg/t3code#7664
* fix(web): copy terminal selection instead of a blank clipboard by @sethwebster in pingdotgg/t3code#7678
* revert(web): restore sparse hourly usage breakdown by @t3-code[bot] in pingdotgg/t3code#7718
* fix(server): reconcile orphaned provider sessions by @t3-code[bot] in pingdotgg/t3code#7719
* fix(web): fix subagent row left border being cut off by @sameerr03 in pingdotgg/t3code#7207
* fix(web): unify composer control rounding by @kototok903 in pingdotgg/t3code#5957

## New Contributors
* @sethwebster made their first contribution in pingdotgg/t3code#7678
* @sameerr03 made their first contribution in pingdotgg/t3code#7207
* @kototok903 made their first contribution in pingdotgg/t3code#5957

**Full Changelog**: pingdotgg/t3code@v0.0.34-nightly.20260820.1142...v0.0.34-nightly.20260820.1146

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.34-nightly.20260820.1146
frankdavidcorona pushed a commit to frankdavidcorona/t3code that referenced this pull request Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants