-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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(type): add Plex Sans Condensed #5680
feat(type): add Plex Sans Condensed #5680
Conversation
Deploy preview for carbon-elements ready! Built with commit 463410b |
Deploy preview for carbon-components-react ready! Built with commit 463410b https://deploy-preview-5680--carbon-components-react.netlify.com |
Deploy preview for carbon-elements ready! Built with commit 3f5cc98 |
Deploy preview for carbon-components-react ready! Built with commit 3f5cc98 https://deploy-preview-5680--carbon-components-react.netlify.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes seem great 🎉 Do we need to add this to the type font family map too or is that not needed?
@joshblack seems like it's been in there, we just weren't declaring the
Or did you mean somewhere else? Not too familiar with the |
Oh perfect! Sounds great |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! 🎉
Closes #5615
Adds in Plex Sans Condensed font-face declarations.
Changelog
New
Changed
Testing / Reviewing
Go to
type/examples
folder, build, then run the example. Ensure that Condensed styles are being shown in all browsers.