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 Tachyons as a CSS framework option #1867

Merged
merged 7 commits into from
Jun 13, 2017
Merged

Conversation

davekiss
Copy link
Contributor

@davekiss davekiss commented Apr 2, 2017

This framework has been picking up some traction, so thought I'd submit a PR to get it implemented in the Sage 9 generator.

http://tachyons.io/

Sage works better without assuming PostCSS is part of the build process. Previously included version implements PostCSS.

See: github.com/tachyons-css/tachyons-sass
@JulienMelissas
Copy link
Contributor

I'd like to put my vote in to approve this one. Love Tachyons.

@Log1x
Copy link
Member

Log1x commented Apr 4, 2017

+1 for Bulma support

@JulienMelissas
Copy link
Contributor

Every single front-end framework/library we add to the Sage repo we need to support minimally. My only issue is when I see Bluma (just as an example) added or requested, it might get to be a bit much. I like Tachyons, but at the same time if everyone wants this that and the other I don't think any of us are seeing the Sage repo be cluttered up. Thoughts?

@joaovpmamede
Copy link

This is a really great add-on.
Btw @davekiss did you try using uncss to get rid of unused css that tachyons might add?

@retlehs
Copy link
Member

retlehs commented Jun 12, 2017

shit @davekiss sorry for letting this sit for so long. can you please fix the merge conflicts? then we'll get this in 👍

@davekiss
Copy link
Contributor Author

@retlehs I believe that should do it, but a quick review would be 👍

@retlehs retlehs merged commit 6516998 into roots:master Jun 13, 2017
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.

5 participants