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

feat: use typography use case tokens in o message #1930

Open
wants to merge 2 commits into
base: 2025-release
Choose a base branch
from

Conversation

frshwtr
Copy link
Contributor

@frshwtr frshwtr commented Jan 16, 2025

Describe your changes

Uses the new typography use case tokens in o-message components

Issue ticket number and link

Link to Figma designs

Checklist before requesting a review

  • I have applied percy label for o-[COMPONENT] or chromatic label for o3-[COMPONENT] on my PR before merging and after review. Find more details in CONTRIBUTING.md
  • If it is a new feature, I have added thorough tests.
  • I have updated relevant docs.
  • I have updated relevant env variables in Doppler.

@frshwtr frshwtr changed the base branch from main to 2025-release January 16, 2025 13:45
font-family: oPrivateFoundationGet('o3-typography-use-case-body-highlight-font-family');
font-size: oPrivateFoundationGet('o3-typography-use-case-body-highlight-font-size');
font-weight: oPrivateFoundationGet('o3-typography-use-case-body-highlight-font-weight');
line-height: oPrivateFoundationGet('o3-typography-use-case-body-highlight-line-height');
}

.o-message__content-main a,
.o-message__content-additional a {
@include oPrivateTypographyLink();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

oPrivateTypographyLink provides text decoration and colour for states for links. Thus why it is still here.

@notlee notlee temporarily deployed to origami-webs-2025-relea-u1mvhl January 16, 2025 13:49 Inactive
@frshwtr frshwtr force-pushed the 2025-release-use-typography-use-case-tokens-in-o-message branch from 3856396 to f0b2fb5 Compare January 16, 2025 13:50
@notlee notlee temporarily deployed to origami-webs-2025-relea-u1mvhl January 16, 2025 13:50 Inactive
@notlee
Copy link
Contributor

notlee commented Jan 16, 2025

Slack conversation here on what to do with the small button: https://financialtimes.slack.com/archives/G03ACR1MJ/p1737037912781139

@frshwtr frshwtr marked this pull request as ready for review January 17, 2025 15:55
@frshwtr frshwtr requested a review from a team as a code owner January 17, 2025 15:55
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