-
Notifications
You must be signed in to change notification settings - Fork 1.2k
sprite: offset-y can take % now #1014
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
Conversation
can you update the test cases also? |
integration tests for the sass extensions are here: https://github.com/chriseppstein/compass/blob/stable/test/integrations/sprites_test.rb |
I have not had time to write tests... I will do my best too soon. |
Would like to see this commit merged |
Awaiting tests before merging |
So this is still a work in progress right? |
I have had no time to spare to get the test suite running on my machine. It On Monday, February 4, 2013, norsewulf wrote:
|
Would love to see this merged. |
This has languished long enough. @scottdavis, can you either accept this patch and add some tests or reject it, please? |
I attempted to run the test suite multiple times and it nearly messed up my On Wednesday, December 4, 2013, Chris Eppstein wrote:
|
I've reopened the feature request. I'm closing this PR because there's been too much change since it was initially submitted. Sorry, We're going to do better going forward. Hopefully, someone can resubmit the change against 1.0. Features should be submitted to the master branch, bug fixes to the stable branch. |
Should be on top of the changes coming in #1771. |
Fixes #1012