Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release/7.0.0 #536

Merged
merged 4 commits into from
Apr 1, 2025
Merged

Release/7.0.0 #536

merged 4 commits into from
Apr 1, 2025

Conversation

georgewrmarshall
Copy link
Contributor

@georgewrmarshall georgewrmarshall commented Apr 1, 2025

Description

This PR introduces version 7.0.0 of the design tokens and corresponding updates to the Tailwind preset. The main changes include:

  1. Breaking changes to the typography system:

    • Removed variant-specific font family classnames (e.g., s-display-md, l-heading-lg)
    • Introduced three base font family classnames:
      • font-default for default text (CentraNo1)
      • font-accent for accent text (MMSans)
      • font-hero for hero text (MMPoly)
    • Updated Body variants (BodyXs, BodySm, BodyMd, BodyLg) font sizes for small screens
  2. Package version updates:

    • @metamask/design-tokens updated to 7.0.0
    • @metamask/design-system-tailwind-preset updated to 0.3.0 (includes breaking changes but because it's in beta we are just bumping the minor version here)
    • Updated peer dependencies to reflect new versions

These changes aim to simplify the typography system while maintaining consistency across MetaMask products. A detailed migration guide has been provided in the design-tokens package.

Related issues

Fixes: NA release PR

Manual testing steps

  1. Review the migration guide in packages/design-tokens/MIGRATION.md
  2. Test typography components using the new font family classnames:
    • Verify font-default renders with CentraNo1
    • Verify font-accent renders with MMSans
    • Verify font-hero renders with MMPoly
  3. Test responsive typography:
    • Check Body variants (BodyXs, BodySm, BodyMd, BodyLg) on small screens
    • Verify font sizes adjust correctly
  4. Verify no regressions in existing typography usage

Pre-merge author checklist

  • I've followed MetaMask Contributor Docs
  • I've completed the PR template to the best of my ability
  • I've included tests if applicable
  • I've documented my code using JSDoc format if applicable
  • I've applied the right labels on the PR (see labeling guidelines)

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed)
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots

Copy link

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: npm/@metamask/[email protected]

View full report↗︎

@georgewrmarshall georgewrmarshall marked this pull request as ready for review April 1, 2025 01:48
@georgewrmarshall georgewrmarshall requested a review from a team as a code owner April 1, 2025 01:48
@georgewrmarshall
Copy link
Contributor Author

georgewrmarshall commented Apr 1, 2025

@metamaskbot publish-preview

@georgewrmarshall georgewrmarshall merged commit 12596ca into main Apr 1, 2025
36 checks passed
@georgewrmarshall georgewrmarshall deleted the release/7.0.0 branch April 1, 2025 15:20
@georgewrmarshall
Copy link
Contributor Author

@metamaskbot publish-preview

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