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(type): match scss definitions to js definitions #5625

Merged
merged 2 commits into from
Mar 20, 2020

Conversation

tw15egan
Copy link
Member

Closes #5548

Matches the scss definitions to the source-of-truth js type definitions

Changelog

Added

  • $expressive-heading-05 should now get light styles at the md breakpoint
  • $expressive-heading-05 and $expressive-heading-06 line-height at max breakpoint

Testing / Reviewing

Ensure nothing is broken, that the scss type definitions match the js definitions

@tw15egan tw15egan requested a review from a team as a code owner March 13, 2020 20:17
@ghost ghost requested review from asudoh and dakahn March 13, 2020 20:18
@netlify
Copy link

netlify bot commented Mar 13, 2020

Deploy preview for carbon-elements ready!

Built with commit 4c06596

https://deploy-preview-5625--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Mar 13, 2020

Deploy preview for carbon-components-react ready!

Built with commit 4c06596

https://deploy-preview-5625--carbon-components-react.netlify.com

Copy link
Contributor

@asudoh asudoh left a comment

Choose a reason for hiding this comment

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

LGTM 👍 - Thanks @tw15egan!

@joshblack joshblack requested review from aagonzales and removed request for jeanservaas March 19, 2020 20:49
Copy link
Member

@aagonzales aagonzales left a comment

Choose a reason for hiding this comment

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

Looks correct!

@joshblack joshblack merged commit 63e2fcc into carbon-design-system:master Mar 20, 2020
renmaddox pushed a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
renmaddox added a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
renmaddox added a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
renmaddox pushed a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
@tw15egan tw15egan deleted the type-updates branch April 28, 2021 18:15
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.

Font Weights not working correctly
5 participants