Skip to content

docs: add CSS utility class guidelines to typography spec - #1371

Open
otsimaofficial wants to merge 1 commit into
SoroScan:mainfrom
otsimaofficial:design/972-typography-css-utility-guidelines
Open

docs: add CSS utility class guidelines to typography spec#1371
otsimaofficial wants to merge 1 commit into
SoroScan:mainfrom
otsimaofficial:design/972-typography-css-utility-guidelines

Conversation

@otsimaofficial

Copy link
Copy Markdown
Contributor

Summary

Closes #972 — typography and hierarchy guidelines for the terminal UI.

docs/design-specs/typography.md already covered font families, the type scale, line-height matrix, color pairings, and Figma text style definitions in detail. This adds the one piece the issue's acceptance criteria called for that was still missing: a CSS utility class guidelines section — explicit do/don't rules for which Tailwind classes map to which hierarchy level (headings, labels, body, badges, etc.), so new components stay consistent with the documented scale instead of introducing one-off sizes/weights.

The Figma text style definitions criterion was already satisfied by the existing §9 ("Figma Font Plugin Setup" / text style definitions in §5) prior to this PR.

Test plan

  • Documentation-only change (docs/design-specs/typography.md); no code paths affected.

🤖 Generated with Claude Code

The typography hierarchy doc already covered fonts, the type scale,
line-height, and Figma text styles. Adds the missing piece from the
issue's acceptance criteria: explicit do/don't guidance for which
Tailwind utility classes to reach for at each hierarchy level, so new
components stay consistent with the existing scale.
@drips-wave

drips-wave Bot commented Aug 28, 2026

Copy link
Copy Markdown

@otsimaofficial Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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