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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(colors): add support for Sass Modules #7395

Merged

Conversation

joshblack
Copy link
Contributor

Adds support for Sass Modules with the colors package through @carbon/colors/scss/module.scss

The generated code for this module looks like:

Screen Shot 2020-12-03 at 15 54 42

This PR also adds in an example for bringing in @carbon/colors to a project and using it as a Sass Module 馃憖

Changelog

New

  • Add module.scss to @carbon/colors

Changed

Removed

Testing / Reviewing

  • Pull down the PR
  • Run yarn build in @carbon/colors
  • Verify module.scss file is built in packages/colors/scss/module.scss
  • Run the example in colors/examples/sass-modules and then run yarn install and yarn develop and verify the color variables work as intended

@joshblack joshblack requested a review from a team as a code owner December 3, 2020 22:17
@netlify
Copy link

netlify bot commented Dec 3, 2020

鉁旓笍 Deploy preview for carbon-elements ready!

馃敤 Explore the source changes: 94b373f

馃攳 Inspect the deploy logs: https://app.netlify.com/sites/carbon-elements/deploys/5fd39f141c1177000870dc67

馃槑 Browse the preview: https://deploy-preview-7395--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Dec 3, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit 94b373f

https://deploy-preview-7395--carbon-components-react.netlify.app

Copy link
Member

@tw15egan tw15egan left a comment

Choose a reason for hiding this comment

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

Everything looks great! Thanks for running through it with me 馃憤

Copy link
Contributor

@aledavila aledavila 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 for the explanation!

@kodiakhq kodiakhq bot merged commit 8ccb1b8 into carbon-design-system:master Dec 11, 2020
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.

None yet

3 participants