Skip to content
This repository was archived by the owner on Jun 20, 2023. It is now read-only.

Switch from compass to node-sass #2482

Merged
merged 13 commits into from
Jul 6, 2016
Merged

Switch from compass to node-sass #2482

merged 13 commits into from
Jul 6, 2016

Conversation

xi
Copy link
Collaborator

@xi xi commented Jul 5, 2016

This pull request removes the dependency on compass, sass, and susy and instead adds dependecy on grunt-sass, node-sass, and sass-planifolia. The resons for that are:

  • node-sass is faster
  • compass is no longer developed
  • we may be able to remove the ruby dependency completely (hologram and and scss-lint still remaining)

Note that sass-planifolia is not a popular library. It is in fact maintained by myself. I think that is fine though as a large part of it is based on the contrast code from a3. So you could think of planifolia as a library that originated from a3 and is now maintained separately.

@xi xi added the frontend label Jul 5, 2016
@joka
Copy link
Collaborator

joka commented Jul 6, 2016

yeah, less ruby more node

@der-john der-john self-assigned this Jul 6, 2016
Conflicts:
	src/adhocracy_frontend/adhocracy_frontend/static/stylesheets/scss/components/_type.scss
@der-john
Copy link

der-john commented Jul 6, 2016

See c498d79: @include inline-block has previously set vertical align in many places. This was not always required. Some other places could also be affected, but this commit covers most of it.

@der-john der-john merged commit 0978910 into master Jul 6, 2016
@der-john der-john removed their assignment Jul 6, 2016
This was referenced Jul 6, 2016
xi pushed a commit that referenced this pull request Aug 17, 2016
regression was introduced in #2482
@joka joka deleted the 2016-07-tb-node-sass branch April 12, 2018 11:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants