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

Added CSS properties for global colours and variables. #26

Merged
merged 4 commits into from
Feb 27, 2020

Conversation

wrux
Copy link
Contributor

@wrux wrux commented Feb 21, 2020

Ticket

Notes

  • I thought about a simple solution that wouldn't require much refactoring. I think we can just create a list of CSS variables from the SASS variables in the _config.core.scss file. With this solution; projects that want to use custom properties can just set the SASS variables to custom property var() values and then the essential global colours and variables are made available at ::root
  • Went with c-- prefx for colours and then g-- prefix for global variables to try and keep the names shorter.
  • May be we can add a variable to conditionally include the custom property file for projects that do not need the functionality.

@colourgarden
Copy link
Contributor

@LucaPipolo I've added a commit to Callum's work to make a couple of necessary changes for it to work in our projects (otherwise we get Gulp errors) and also to bump the version number so we can publish it to npm.

Could you take a look please?

@LucaPipolo LucaPipolo merged commit 0330ddf into master Feb 27, 2020
@LucaPipolo LucaPipolo deleted the feature/css-properties-callum branch February 27, 2020 09:51
colourgarden added a commit that referenced this pull request May 14, 2020
Incorrectly removed in #26.
Though it is no longer used in Bacon itself, some projects rely on this mixin.
colourgarden added a commit that referenced this pull request May 14, 2020
Incorrectly removed in #26.
Though it is no longer used in Bacon itself, some projects rely on this mixin.
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.

3 participants