improvement(ui): consolidate product typography - #8132
BillLeoutsakosvl346 wants to merge 5 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
There was a problem hiding this comment.
All reported issues were addressed across 58 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |
Comments Outside DiffThese findings sit on lines the diff does not cover, so they could not be posted inline. Each one leaves this list once its file changes.
|
|
The global-scale changes are an explicit, approved part of this typography consolidation. The requested implementation centralizes shared sizes, weights, and the existing scoped Inter utility in The imported 14/20/24px rem values and their default leading are preserved. The new 28/32/40px tokens represent existing repeated heading roles, and the other visual size/weight consolidations are described in the PR. Inter is still loaded only within its existing boundary. These are deliberate central definitions, not accidental global overrides, so no code change is appropriate for the outside-diff finding. |
|
Representative before/after comparisons from the real application, using the same viewport, canvas scale, theme, and control state.
The close-ups below are enlarged from retained original screenshots. |
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |

Summary
cnthe new size names.Type of Change
Testing
Checklist