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

Add rudimentary SASS integration. #77

Merged
merged 1 commit into from Oct 2, 2015
Merged

Add rudimentary SASS integration. #77

merged 1 commit into from Oct 2, 2015

Conversation

toolness
Copy link
Contributor

@toolness toolness commented Oct 2, 2015

Since @claudioccm's CSS is SASS-based, it'd be really nice if we could have the source SASS in this repository and easily iterate on it from here.

This PR adds libsass-python to our app container and integrates SASS into our build process.

For development, we use SASS middleware that regenerates the SASS on each request and gives us handy error feedback in our views like this:

2015-10-02_12-56-32

For production, I've added manage.py build_sass, which is executed as part of run.sh, so that our compiled SASS is served as static CSS in production.

We also compile our SASS as part of our test suite to make sure it's always syntactically correct.

@@ -1,6 +1,7 @@
.env
*.pyc
app/config/local_config.yml
app/static/css
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is where our generated SASS goes... It's confusing right now because we also have hand-written CSS in app/static/styles, but once we've fully moved over to claudio's CSS, I assume we'll get rid of that.

talos added a commit that referenced this pull request Oct 2, 2015
Add rudimentary SASS integration.
@talos talos merged commit ea751a7 into master Oct 2, 2015
@talos talos deleted the sass branch October 2, 2015 19:23
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.

None yet

2 participants