Skip to content

fix(web): polish interface spacing - #5252

Merged
maria-rcks merged 1 commit into
mainfrom
agent/polish-interface-spacing
Aug 3, 2026
Merged

fix(web): polish interface spacing#5252
maria-rcks merged 1 commit into
mainfrom
agent/polish-interface-spacing

Conversation

@maria-rcks

@maria-rcks maria-rcks commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • align composer, sidebar, and top-bar control insets
  • size the context-window ring consistently with the send control
  • use the shared bordered glass treatment for context-window details

Why

Several nearby controls used slightly different hardcoded insets and the context-window popover overrode the shared floating-surface styling. This made edges look uneven and the context control feel inconsistent with the rest of the interface.

Validation

  • vp fmt --check on the eight changed web files
  • vp run --filter @t3tools/web typecheck

Created with GPT-5.6 Sol via the Codex harness.

Note

Polish interface spacing across web UI components

Makes minor padding and spacing adjustments across several components for visual consistency. Changes affect BranchToolbar, ChatComposer, GitActionsControl, OpenInPicker, SidebarV2, SettingsSidebarNav, and ContextWindowMeter. The ContextWindowMeter popover also gains stroke color transition animation on the SVG circle alongside the existing dash offset animation.

Macroscope summarized 4a87482.


Note

Low Risk
CSS-only spacing and popover styling tweaks with no logic, auth, or data-path changes.

Overview
Aligns horizontal insets across the composer strip, sidebar rows, header icon buttons, and workspace titlebar so neighboring controls share the same visual edge.

Composer & branch contextBranchToolbar uses asymmetric ps-1 pe-2 instead of uniform horizontal padding. ChatComposer bumps footer padding one step (px-3/pb-3, sm:px-4/sm:pb-4) and nudges the model picker trigger (-ms-px ps-0).

Sidebar & settingsSidebarV2 offsets the project filter trigger and thread list group with shared inset tokens (--sidebar-row-content-inset, --sidebar-content-inset). SettingsSidebarNav adds a 1px start padding on the menu list.

Header actionsGitActionsControl and OpenInPicker apply ps-[8.5px] on xs outline buttons so icon-leading controls line up with peers. ChatView adds mr-px on workspace titlebar controls.

Context window meter — Simplifies ring strokeDashoffset math, extends the SVG transition to include stroke, and switches the details popover to shared floating-surface styling (viewportClassName="p-0", p-[var(--floating-content-inset)]) instead of custom glass overrides.

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

@coderabbitai

coderabbitai Bot commented Aug 3, 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: 521cdbb4-41df-4f27-b6ca-f50407f84ca1

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:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Aug 3, 2026
@maria-rcks
maria-rcks marked this pull request as ready for review August 3, 2026 02:49

@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.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1f8ff10. Configure here.

Comment thread apps/web/src/components/chat/ContextWindowMeter.tsx Outdated
macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Aug 3, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 4a87482

All changes are CSS class modifications for spacing and padding adjustments across UI components. No logic or behavioral changes - purely visual polish with minimal risk.

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

@maria-rcks
maria-rcks force-pushed the agent/polish-interface-spacing branch from 1f8ff10 to 0c68a95 Compare August 3, 2026 02:58
@macroscopeapp
macroscopeapp Bot dismissed their stale review August 3, 2026 02:58

Dismissing prior approval to re-evaluate 0c68a95

macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Aug 3, 2026
@maria-rcks
maria-rcks force-pushed the agent/polish-interface-spacing branch from 0c68a95 to 4a87482 Compare August 3, 2026 03:22
@macroscopeapp
macroscopeapp Bot dismissed their stale review August 3, 2026 03:22

Dismissing prior approval to re-evaluate 4a87482

@github-actions github-actions Bot added size:S 10-29 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Aug 3, 2026
@maria-rcks
maria-rcks merged commit 30c9622 into main Aug 3, 2026
17 checks passed
@maria-rcks
maria-rcks deleted the agent/polish-interface-spacing branch August 3, 2026 03:35
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 3, 2026
stevesarmiento added a commit to stevesarmiento/harness that referenced this pull request Aug 3, 2026
…nus, server fixes

25 upstream commits (through 6f04a5c): settings sidebar search (pingdotgg#4682),
sidebar thread search (pingdotgg#4769), libghostty-vt terminal renderer (pingdotgg#4860,
pingdotgg#5102), legacy-model menu folding (pingdotgg#5190), npx t3 pair QR (pingdotgg#4955), bounded
catch-up replay (pingdotgg#5147), rollback-safe updates (pingdotgg#5181/pingdotgg#5095/pingdotgg#5217), PR
branch-drift threading (pingdotgg#5159), no auto-settle with open PRs (pingdotgg#5151),
interface spacing polish (pingdotgg#5252), and fixes.

Forma preserved: composer 40px squircle, sidebar chrome + separator
removal, Forma icon art, index.css sections (pixel grid et al), desktop
identity + ~/.forma default (now via DesktopStatePaths helper), settings
IA (nav renders Forma sections; upstream search indexes both IAs, fork
sections registered in settingsSearch), GitActions commit dialog,
ghostty terminal chrome colors generated from the Forma theme with
setTheme observers and code-font sizing intact.

Deferred: MobileRunContextSelector (BranchToolbar rename side), upstream
quick-action popover styling in GitActionsControl.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant