Skip to content

design-audit: share phoneBarTitleSx for the phone batch top-bar titles - #1023

Open
frewsxcv wants to merge 1 commit into
mainfrom
claude/keen-goodall-t75voh
Open

frewsxcv wants to merge 1 commit into
mainfrom
claude/keen-goodall-t75voh

Conversation

@frewsxcv

@frewsxcv frewsxcv commented Nov 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • PhoneStep's header title (17px/700) and PhoneGrid's selection-mode title (18px/700) were the same role with drifted magic numbers.
  • Adds phoneBarTitleSx to common/layoutSx.ts and uses it in both. The selection title goes from 18px to 17px so the two top bars match.

Verification

  • oxfmt --check and oxlint pass on the changed files.
  • Full tsc couldn't run because dependencies aren't installed in the sandbox. It's a pure refactor of sx values.

Generated by Claude Code

…s (17px vs 18px drift)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WA2AWi6uXnyKxv5mEQsZnT

This branch has not been deployed

No deployments
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.

2 participants