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 SASS version #1227

Merged
merged 3 commits into from
Dec 15, 2015
Merged

add SASS version #1227

merged 3 commits into from
Dec 15, 2015

Conversation

bsorem-sugarcrm
Copy link

Answers #1199

@shehi
Copy link

shehi commented Dec 13, 2015

Can we please get this merged asap? I need SASS as well. Thanks!

@dhuf
Copy link

dhuf commented Dec 15, 2015

+1 for me

@shehi
Copy link

shehi commented Dec 15, 2015

Ping @silviomoreto @caseyjhol @t0xicCode

@silviomoreto
Copy link
Collaborator

@bsorem-sugarcrm I cant test your commit. Can I trust in your PR?

silviomoreto added a commit that referenced this pull request Dec 15, 2015
@silviomoreto silviomoreto merged commit 4d47d75 into snapappointments:master Dec 15, 2015
@bsorem-sugarcrm
Copy link
Author

@silviomereto I hope so! I compiled it to CSS using SASS and manually compared that to the LESS-generated CSS.

@silviomoreto
Copy link
Collaborator

Ok, it is already accepted. Thanks

@shehi
Copy link

shehi commented Dec 16, 2015

Dear @silviomoreto , even if it has problems, we - SASS users - will send additional PRs to fix it up :D Thanks for the merge. Please make sure you have a release with this inside, so that we could use it with Bower.

@bsorem-sugarcrm : at least you verified it's integrity by converting to CSS right (with Compass or some JS/Gulp tool)?

@silviomoreto
Copy link
Collaborator

done 4d47d75

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.

4 participants