8000 fix(tokens): pull in changes to font weight tokens by mcoker · Pull Request #6884 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

fix(tokens): pull in changes to font weight tokens#6884

Merged
mcoker merged 3 commits intopatternfly:v6from
mcoker:import-tokens-7.12.24
Jul 12, 2024
Merged

fix(tokens): pull in changes to font weight tokens#6884
mcoker merged 3 commits intopatternfly:v6from
mcoker:import-tokens-7.12.24

Conversation

@mcoker
Copy link
Contributor
@mcoker mcoker commented Jul 12, 2024

fixes #6883

A backstop run (using the desktop viewport size) before/after these changes showed 0 differences

@patternfly-build
Copy link
Collaborator
patternfly-build commented Jul 12, 2024

@mcoker mcoker requested review from srambach and thatblindgeye July 12, 2024 16:56
Copy link
Contributor
@thatblindgeye thatblindgeye left a comment

Choose a reason for hiding this comment

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

This looks good, only thing is should the bold token be removed from the local file?

--pf-t--global--font--weight--body--bold: var(--pf-t--global--font--weight--200);
--pf-t--global--font--weight--heading--bold: var(--pf-t--global--font--weight--400);

@mcoker
Copy link
Contributor Author
mcoker commented Jul 12, 2024

@thatblindgeye d'oh!! Yep, thanks for catching that, didn't check in that change for some reason. Updated 👍

@mcoker mcoker merged commit 0d6a2a1 into patternfly:v6 Jul 12, 2024
@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 6.0.0-alpha.189 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0