Skip to content

Improve comment readability on large displays - #150

Open
AndySparks wants to merge 1 commit into
Lex-Inc:mainfrom
AndySparks:fix/wide-review-rail
Open

AndySparks wants to merge 1 commit into
Lex-Inc:mainfrom
AndySparks:fix/wide-review-rail

Conversation

@AndySparks

@AndySparks AndySparks commented Sep 4, 2026 •

Copy link
Copy Markdown

Summary

  • Keep the writing canvas at its existing 46.5rem measure.
  • Let the review rail grow from 18rem to 30rem as larger displays provide room.
  • Cap comment text near a 65-character measure after accounting for padding and the avatar gutter.
  • Add a browser regression covering both the unchanged writing canvas and responsive comment rail.

Why

Long review comments currently remain in an 18rem rail even on very large displays, producing a tall, narrow column while substantial horizontal space is unused. This makes comments easier to read without changing the writing experience.

Browser measurements

Viewport Writing canvas Comment rail
1440px 744px 288px
2560px 744px 480px

Verification

  • pnpm check
  • pnpm test:smoke — 12 passed
  • pnpm test:coverage
  • Focused Playwright viewport regression

GitHub Actions is awaiting first-time contributor approval. The same CI commands passed locally under the repository's pnpm 10 environment. The Vercel preview also requires Lex team authorization for external forks.

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

@AndySparks is attempting to deploy a commit to the Lex Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant