Skip to content

refactor: use system-level typography #31334

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

Merged
merged 6 commits into from
Jun 13, 2025

Conversation

andrewseguin
Copy link
Contributor

@andrewseguin andrewseguin commented Jun 11, 2025

Aligns typography with M3 system terminology and moves it to the system mapping for consistency with M2 color tokens and all M3 tokens

Right now, this config mapping has been set up as the following:

M3 System M2 Config
body-large body-1
body-medium body-2
body-small caption
label-large subtitle-1
label-medium subtitle-2
label-small button
display-large headline-1
display-medium headline-1
display-small headline-1
headline-large headline-2
headline-medium headline-3
headline-small headline-4
title-large headline-5
title-medium headline-5
title-small headline-6

@andrewseguin andrewseguin requested a review from a team as a code owner June 11, 2025 15:37
@andrewseguin andrewseguin requested review from crisbeto and mmalerba and removed request for a team June 11, 2025 15:37
@andrewseguin andrewseguin added the target: minor This PR is targeted for the next minor release label Jun 11, 2025
Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there's a build error

@andrewseguin andrewseguin force-pushed the sys-typography-2 branch 2 times, most recently from a9a61f0 to a91c1d7 Compare June 12, 2025 22:02
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Jun 13, 2025
@andrewseguin andrewseguin merged commit e0bc165 into angular:main Jun 13, 2025
22 of 25 checks passed
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jul 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants