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

fix(tokens)!: fix all letter spacing values #537

Merged
merged 6 commits into from
Sep 10, 2024

Conversation

Natalie9
Copy link
Contributor

@Natalie9 Natalie9 commented Aug 16, 2024

Infos

Task

What is being delivered?

The value of the letter-spacing variables was incorrectly converted from % to em. The corresponding correction was made, where

16% === 0.16em
4% === 0.04 em

What impacts?

All fonts in all applications.

Reversal plan

Rollback this PR

Evidences

Before

letter-spacing: 0.0004em == 0.04%
image

After

letter-spacing: 0.04em == 4%
image

Desing System

image

@Natalie9 Natalie9 self-assigned this Aug 16, 2024
Copy link
Contributor

github-actions bot commented Aug 16, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-09-10 12:38 UTC

felipefialho
felipefialho previously approved these changes Aug 16, 2024
lupy100
lupy100 previously approved these changes Aug 16, 2024
mauriciomutte
mauriciomutte previously approved these changes Aug 16, 2024
@felipefialho felipefialho changed the title fix(tokens): fix all letter spacing values fix(tokens)!: fix all letter spacing values Aug 16, 2024
tassioFront
tassioFront previously approved these changes Aug 16, 2024
gabrielduete
gabrielduete previously approved these changes Aug 16, 2024
felipefialho
felipefialho previously approved these changes Aug 19, 2024
mauriciomutte
mauriciomutte previously approved these changes Aug 20, 2024
packages/tokens/package.json Outdated Show resolved Hide resolved
packages/tokens/package.json Outdated Show resolved Hide resolved
@Natalie9 Natalie9 force-pushed the fix/tokens-letter-spacing-value branch 2 times, most recently from 1d6a995 to e1aa18c Compare August 30, 2024 08:51
@Natalie9 Natalie9 merged commit 550fd34 into main Sep 10, 2024
2 checks passed
@Natalie9 Natalie9 deleted the fix/tokens-letter-spacing-value branch September 10, 2024 12:35
This was referenced Sep 10, 2024
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.

7 participants