Skip to content

Composer | Use 18px title - #57

Merged
BrianGenisio merged 1 commit into
mainfrom
feature/composer-title-18px
Oct 15, 2026
Merged

BrianGenisio merged 1 commit into
mainfrom
feature/composer-title-18px

Conversation

@BrianGenisio

@BrianGenisio BrianGenisio commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

The compose panel title was 15px body-small. Bump it to 18px so it reads as a heading.

Changes

The typography scale has no 18px heading token; 16px and 22px are the nearest. .composer__title sets 18px locally and still uses --heading-family for Founders Grotesk.

image

Test plan

  • Open Compose and confirm "New message" is 18px Founders Grotesk
  • Open a reply and confirm the title size matches

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: c2e94cf9-ac04-49e0-ae6c-780a71d9c3e6
📥 Commits

Reviewing files that changed from the base of the PR and between 1273d16 and 093d6f6.

📒 Files selected for processing: 1
  • public/app.css

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

📝 Walkthrough

The .composer__title rule now uses the heading-family font at 18px. Its line height changes from 1.35 to 1.2, and its letter spacing is set to −0.18px.

Suggested reviewers: aimemp

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 093d6

The composer title changes to 18px, with no established clipping or overlap in the inspected layouts. The exact runtime font remains unverified, but no actionable merge-blocking risk is established.

Pre-merge checks: Passed 4
Passed Passed checks: 4 passed
Check name Status Explanation
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Title check Passed The title clearly and concisely describes the main change: updating the composer title to 18px.
Description check Passed The description directly explains the typography change, implementation, and test plan for the composer title.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@BrianGenisio
BrianGenisio merged commit e13b877 into main Oct 15, 2026
2 checks passed
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.

1 participant