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

Begin applying use-case tokens to o3 components. #1925

Merged
merged 4 commits into from
Jan 15, 2025

Conversation

notlee
Copy link
Contributor

@notlee notlee commented Jan 14, 2025

Builds on #1923 with:

  • Remaining heading tokens
  • Quote tokens

Note that o3-editorial-typography-pullquote-content tokens have the wrong values – this needs to be updated in Token Studio and applied in Figma. Feedback left for the design team. Moving on for now.

@notlee notlee requested a review from a team as a code owner January 14, 2025 15:04
Base automatically changed from 2025-release-apply-type-usecases-o3 to 2025-release January 14, 2025 16:04
@notlee notlee changed the base branch from 2025-release to main January 14, 2025 16:04
@notlee notlee changed the base branch from main to 2025-release January 14, 2025 16:05
o3-editorial-typography-pullquote-content have the wrong values – this needs to be updated in Token Studio and applied in Figma. Feedback left for the design team. Moving on for now.
@notlee notlee force-pushed the 2025-release-apply-type-usecases-o3-2 branch from 9bd1a78 to d08e6e6 Compare January 14, 2025 16:06
@notlee notlee temporarily deployed to origami-webs-2025-relea-gss413 January 14, 2025 16:11 Inactive
font-size: var(--o3-typography-use-case-label-font-size);
font-weight: var(--o3-typography-use-case-label-font-weight);
line-height: var(--o3-typography-use-case-label-line-height);
text-transform: var(--o3-typography-use-case-label-text-case);
Copy link
Contributor

Choose a reason for hiding this comment

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

This token is undefined, is this expected?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good spot, I think you mean --o3-typography-use-case-detail-text-case? I've removed that

@notlee notlee requested a review from frshwtr January 15, 2025 15:32
@notlee notlee temporarily deployed to origami-webs-2025-relea-gss413 January 15, 2025 15:32 Inactive
@notlee notlee merged commit 536e81d into 2025-release Jan 15, 2025
4 checks passed
@notlee notlee deleted the 2025-release-apply-type-usecases-o3-2 branch January 15, 2025 16:11
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