You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 11, 2021. It is now read-only.
With the work on the Buttons PR #97 and the Typography Variable Refactoring PR #134, it has become obvious that we need to create a style guide for how to best handle JSASS variables in Chassis, including naming conventions, and when to group variables under a larger object. Lets use this issue as a starting point for best practices we want to follow.
For example, a good practice to include is probably to group variables that belong to the same element together. Also we should have some consistency between using camel case for variables or dashes. If anyone else can think of anything else requiring consistency that we should address, please add it here.
The text was updated successfully, but these errors were encountered:
Paraphrasing today's discussion in the weekly Chassis Meeting: SASS style guide will mention that we use JSASS variables instead of traditional SCSS Variables, then link to the information for JSASS. Also it would be useful to have some explanation on how JSASS works and integrating it with your development process.
With the work on the Buttons PR #97 and the Typography Variable Refactoring PR #134, it has become obvious that we need to create a style guide for how to best handle JSASS variables in Chassis, including naming conventions, and when to group variables under a larger object. Lets use this issue as a starting point for best practices we want to follow.
For example, a good practice to include is probably to group variables that belong to the same element together. Also we should have some consistency between using camel case for variables or dashes. If anyone else can think of anything else requiring consistency that we should address, please add it here.
The text was updated successfully, but these errors were encountered: